[MPlayer-dev-eng] the great reformatting

Ingo Brückl ib at wupperonline.de
Tue Feb 15 11:00:17 CET 2011


Clément Boesch wrote on Tue, 15 Feb 2011 08:17:16 +0100:

> What I fear the most is the vertical alignment we can find in various
> places, here for instance:

>     100.0 * video_time_usage / total_time_usage,
>     100.0 * vout_time_usage / total_time_usage,
>     100.0 * audio_time_usage / total_time_usage,

> This could have been:

>     100.0 * video_time_usage / total_time_usage,
>     100.0 *  vout_time_usage / total_time_usage,
>     100.0 * audio_time_usage / total_time_usage,

Your example is safe since sp_arith is add, thus not changing an existing
vertical alignment if it at least follows the rule to have one space between
the arithmetic operators.

The problem rather is (and please forgive me to mention this again) if you
have to add a

  100.0 * decoding_time_usage / total_time_usage.

You have to needlessly change the whole block's alignments.

Ingo


More information about the MPlayer-dev-eng mailing list