Re: RFC: comctl32 indentations

2008-05-25 Thread Hin-Tak Leung
Andrew Talbot wrote: > If, in this case, it turns out that the braces should be added, and doing so > would fix some longstanding bug with Photoshop CS2, say, then I doubt that > too many people would accuse me of having wasted their time. This is still a waste of time - I am not going to talk

Re: RFC: comctl32 indentations

2008-05-24 Thread Hin-Tak Leung
Andrew Talbot wrote: > Andrew Talbot wrote: > >> If the forum is the wrong place to raise this sort of query, please >> forgive and advise. :) >> > > Actually, Alexandre suggested that I file bug reports for things I find but > can't fix myself. And I suppose an indentation anomaly is still a sor

Re: RFC: comctl32 indentations

2008-05-24 Thread Andrew Talbot
Andrew Talbot wrote: > If the forum is the wrong place to raise this sort of query, please > forgive and advise. :) > Actually, Alexandre suggested that I file bug reports for things I find but can't fix myself. And I suppose an indentation anomaly is still a sort of bug(?) -- Andy.

RFC: comctl32 indentations

2008-05-24 Thread Andrew Talbot
Hi, In toolbar.c:TOOLBAR_Destroy(), should the "if" statement at line 5439 be compound to match the indentation, or should the three invocations of TOOLBAR_DeleteImageList() be outdented? treeview.c: In TREEVIEW_DeleteItem(), how conditional is the call of TREEVIEW_SetFirstVisible() at line 15