Code review comment for lp://staging/~abreu-alexandre/ubuntu-html5-theme/control-automatic-orientation-of-app

Revision history for this message
David Barth (dbarth) wrote :

Le 25/02/2014 14:35, Alexandre Abreu a écrit :
>> Feels like a race condition solved by a different initialization path.
>>
>> I'm not sure the command line option really makes sense but for tests though:
>> can you remove it? less code is better.
> Not sure I understand the comment, ... this is a new command line arg & option
> ... that's it,
right, but less is better: more options means more code and backward
compatibility overhead in the future.

David

« Back to merge proposal