Hi Miklos, Done. The updated patch is here: https://sourceforge.net/p/libepubgen/tickets/_discuss/thread/d1bdd53a/fe73/attachment/0001-Replace-assertCss-with-CPPUNIT_ASSERT_CSS.patch
2018-04-03 21:18 GMT+08:00 Miklos Vajna <[email protected]>: > Hi Mark, > > Cppunit suggests to write your own assert macros if you want to preserve > line number information. See libreoffice core.git > sw/qa/extras/inc/swmodeltestbase.hxx:965 for an example. > > Do you think you could rework your patch, so that you add a new > CPPUNIT_ASSERT_CSS() (or similar) macro, rather than use exceptions? > > Thanks, > > Miklos > -- Mark Hung -- To unsubscribe e-mail to: [email protected] Problems? https://www.libreoffice.org/get-help/mailing-lists/how-to-unsubscribe/ Posting guidelines + more: https://wiki.documentfoundation.org/Netiquette List archive: https://listarchives.documentliberation.org/www/devel/ All messages sent to this list will be publicly archived and cannot be deleted
