[MPlayer-dev-eng] [PATCH] fix dvdnav multi-cell menus

Carl Eugen Hoyos cehoyos at ag.or.at
Wed Aug 17 14:27:30 CEST 2011


Steaphan Greene <sgreene <at> cs.binghamton.edu> writes:

> As for the separation of white-space changes, does that really include
> cases like this, where the white-space changes are a direct result of a
> functional change (removal the else {} block)?

It could even be argued that the case you describe is the only reason for the
rule in question.
As far as MPlayer and FFmpeg are concerned, you may leave the re-indentation to
the committer (although a second whitespace patch doesn't hurt).

Carl Eugen



More information about the MPlayer-dev-eng mailing list