Dirkjan Ochtman <dirkjan@ochtman.nl> [Fri, 08 May 2009 22:35:10 +0200] rev 8326
merge with mpm
Nicolas Dumazet <nicdumz.commits@gmail.com> [Mon, 04 May 2009 18:04:41 +0900] rev 8325
inotify: inotify.server.walk() simplify control flow
Nicolas Dumazet <nicdumz.commits@gmail.com> [Mon, 04 May 2009 17:58:26 +0900] rev 8324
inotify: inotify.server.walk*() remove unnecessary var
Remove hginside var and the test it relates to:
not( top or not hginside ) == (not top) and hginside, so the only case
when nothing will be yielded is when hginside is True and top is False.
Because of the returns placed upstream, this case will not happen anymore.
We can then safely remove hginside and the (if)s
Nicolas Dumazet <nicdumz.commits@gmail.com> [Mon, 04 May 2009 17:17:03 +0900] rev 8323
inotify: inotify.server.walk*() cleanup
When not in root repo, if we meet a .hg, bail out.
break was used, but return can be safely used instead: if we go through break
then (top or not hginside) == False -> nothing is done after the for loop
Nicolas Dumazet <nicdumz.commits@gmail.com> [Mon, 04 May 2009 17:11:49 +0900] rev 8322
inotify: inotify.server.walkrepodirs() simplify walking
Do not 'yield tuple, boolean' to filter later on the boolean.
Test the boolean first, and yield tuple only if needed.
Nicolas Dumazet <nicdumz.commits@gmail.com> [Mon, 04 May 2009 17:06:59 +0900] rev 8321
inotify: inotify.server.walkrepodirs() simplify
compute string concatenation only once