Stefan Beller <sbel...@google.com> writes: > On Thu, Mar 16, 2017 at 3:29 PM, Brandon Williams <bmw...@google.com> wrote: >> Sync does some work determining what URLs should be used for a submodule >> but then throws this work away if the submodule isn't active. Instead >> perform the activity check earlier and skip inactive submodule in order >> to avoid doing unnecessary work. >> >> Signed-off-by: Brandon Williams <bmw...@google.com> >> --- > >> + # skip inactive submodules >> + if ! git config "submodule.$name.url" >/dev/null 2>/dev/null
Just a style thing, but ">/dev/null 2>&1" is probably shorter and more idiomatic.
