[PATCH] winecfg: Double click in dlls list to edit overides; fix indent. (try 2)

Andrew Eikum aeikum at codeweavers.com
Tue Feb 10 08:04:27 CST 2015


On Fri, Feb 06, 2015 at 10:23:15AM +0800, Hao Peng wrote:
> Add a feature to winecfg. User can double click in dlls list to edit overides of selected item.
> 
> the indent of code here mixed with tabs, 4 spaces and 8 space. I formated it to match the style of the rest code.
> 

Usually we don't take patches that change nothing except formatting.

I think your first patch was rejected because it adds even more
inconsistent formatting. The indent is too small. The "case
CBN_DBLCLK" line should be indented the same as the "case
LBN_SELCHANGE" line. Instead it's indented 8 spaces when it should be
indented 16 spaces.

Andrew



More information about the wine-devel mailing list