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


At 7:24pm -0400 Mon, 24 Oct 2011, Fridrich Štrba wrote:
New commits:
commit 38cf494f90d19d1673773437cd52e8f0cfbf4eb5
Author: Fridrich Štrba <fridrich.strba@bluewin.ch>
Date:   Tue Oct 25 01:11:49 2011 +0200

    White-space change to fix messy mix of different coding styles

[...]

diff --git a/writerperfect/source/filter/FontStyle.cxx b/writerperfect/source/filter/FontStyle.cxx
index 5743e1e..ac501c6 100644

  void FontStyleManager::clean()
  {
      for (std::map<WPXString, FontStyle *, ltstr>::iterator iter = mHash.begin();
-         iter != mHash.end(); ++iter)
+            iter != mHash.end(); iter++)
      {

@@ -65,7 +63,7 @@ void FontStyleManager::writeFontsDeclaration(OdfDocumentHandler *pHandler) const

      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


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.