On Fri, 2010-11-19 at 12:09 -0800, Kayo Hamid wrote: > Sending for review. It covers source/filter/ dirs. > >
I kept stuff like // 0.01mm = 0.57twip as that's a useful comment IMO and I kept stuff like void SwHTMLParser::EndNumBulListItem( int nToken, sal_Bool bSetColl, - sal_Bool /*bLastPara*/ ) + sal_Bool ) as that's a common enough pattern which I don't have a problem with where the argument is commented out to avoid a compiler warning when someone has to override a method but doesn't need a particular argument in the overridden method, and it helps comment what that other argument is for. Otherwise, happy with the rest of it and pushed it now. Thanks for this. C. _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice