Re: [www-devel] patch: extract EPUBBodyStyleManager from EPUBSpanStyleManager

Hi Mark,

On Fri, Apr 20, 2018 at 10:28:51PM +0800, Mark Hung <marklh9@gmail.com> wrote:
> Here are two patches related to extract EPUBBodyStyleManager from
> EPUBSpanStyleManager: https://sourceforge.net/p/libepubgen/tickets/5/

Please don't forget to add doxygen documentation for the new code. At a
minimum, if you add a new class (EPUBBodyStyleManager in your case),
please add a one-liner mission statement for it:

/// Foo bar.
class Foo
{
};

It's trivial to do it now, and it'll help half year later when you
forget things. :-)

Thanks,

Miklos

-- 
To unsubscribe e-mail to: devel+unsubscribe@documentliberation.org
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