Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Actually, the post/preincrement change was an accidental one. The
problem I have is that the writeperfect source code lives in two
different places. The original code lives in
sourceforge.new/projects/libwpd as writerperfect module. That one
produces standalone converters for all the file-format. And that is
the place where it all started, kind of historical original place. I
normally fix that one and then use the same patch to patch the
libreoffice tree.

It is a bit hard for me to sync those things when the change is done
only in libreoffice and some larger changes are difficult to sync
using patches, so I just copy the corresponding classes from the
SF.net project and reformat the code according to the libreoffice
coding standards. This pre/postincrement change did not happen in the
sf.net repo and during the sync got lost. Although, just in the for
loops, I don't see much difference in the two.

I am thinking about making the writerperfect generators just a library
and use it as any other external library inside libreoffice build. But
that is not for tomorrow.

Cheers

F.

On 25/10/11 04:04, Kevin Hunter wrote:
At 9:48pm -0400 Mon, 24 Oct 2011, Kevin Hunter wrote:
At 7:24pm -0400 Mon, 24 Oct 2011, Fridrich Štrba wrote:
for (std::map<WPXString, FontStyle *, ltstr>::const_iterator
iter = mHash.begin(); - iter != mHash.end(); ++iter) + iter !=
mHash.end(); iter++)

Err, in terms of coding style, is a pre to post increment
operator merely a whitespace change? I seem to recall a message
or two on the subject but I'm having a difficult time tracking
them down just now. About 8 months ago, perhaps?

Kevin: open mouth, insert foot.  I apologize.  Just perusing
commits, should've kept going.  I take it the pre to post is needed
to "fix the build".

Kevin


-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.16 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk6me3AACgkQu9a1imXPdA+rRwCdFxwc/C6abM2x8gQ8yo1EqHhg
MHMAnA+MiUxLe1EFJCQyh2heenV0CmMw
=fnJH
-----END PGP SIGNATURE-----

Context


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.