[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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. :-)



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

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.