<table cellspacing="0" cellpadding="0" border="0" ><tr><td valign="top" style="font: inherit;"><br>Thanks for your review, I'll take into account in the next patch.<div><br></div><div>revol_<br><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;"><div class="plainMail">I kept stuff like<br>// 0.01mm = 0.57twip <br>as that's a useful comment IMO<br><br>and I kept stuff like<br><br> void SwHTMLParser::EndNumBulListItem( int nToken, sal_Bool bSetColl, <br>-&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; sal_Bool /*bLastPara*/ ) <br>+&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; sal_Bool ) <br><br>as that's a common enough pattern which I don't have a problem with<br>where the argument is commented out to avoid a compiler warning when<br>someone
 has to override a method but doesn't need a particular argument<br>in the overridden method, and it helps comment what that other argument<br>is for.<br><br>Otherwise, happy with the rest of it and pushed it now. Thanks for this.<br><br>C.<br></div></blockquote></div></td></tr></table><br>



      &nbsp;