<div dir="ltr"><br>Hi Miklós,<br><div class="im"><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

- assertEqualBorder() -> CPPUNIT_ASSERT_EQUAL_BORDER(): in C++, in<br>
  general inline functions are preferred over macros. If the intention<br>
  is to get the place where the real problem is, run the unit test in<br>
  gdb, and get a backtrace.<br></blockquote><div><br></div></div><div>I looked up how to write new unit test macros<br><a href="http://cppunit.sourceforge.net/doc/lastest/group___assertions.html" target="_blank">http://cppunit.sourceforge.net/doc/lastest/group___assertions.html</a> and<br>

<a href="http://cppunit.sourceforge.net/doc/lastest/struct_asserter.html" target="_blank">http://cppunit.sourceforge.net/doc/lastest/struct_asserter.html</a><br>and
 rewrite the mentioned macro using an inline function inside it, so it 
becomes more safe, which is the biggest advantage of using functions 
instead of macros. Plus with it, the assertion will point to the right 
place.<br>
I pushed the diff to gerrit, if you have other comments to it:<br><a href="https://gerrit.libreoffice.org/#/c/5885/" target="_blank">https://gerrit.libreoffice.org/#/c/5885/</a><br></div><div class="im"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


- If you deal with colors in unit tests, I think writing them in hex<br>
  form is much more readable (16711680 vs 0xff0000). Or in case it's a<br>
  common color, even better: COL_RED<br></blockquote><div><br></div></div><div>It
 seems in LO souce code COL_RED not equal with 0xff0000. I tried to use 
it in the unit tests, but they failed. So I keep at hex form. <br></div><div class="im"><div>
 </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
- sw/source/filter/ww8/attributeoutputbase.hxx: Please avoid using<br>
  'using' in header files, we only do that in cxx files, hxx files have<br>
  all the namespaces written out for better readability.<br></blockquote><div> <br></div></div><div>Ok,<br><br></div><div>Best regards,<br></div>Tamás</div>