[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdepim/plugins/kmail/bodypartformatter
From:       Till Adam <adam () kde ! org>
Date:       2008-02-17 12:22:30
Message-ID: 1203250950.065098.13899.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 776105 by tilladam:

Missed one QString, make them static, as per Ingo's suggestion.


 M  +7 -7      text_xdiff.cpp  


--- trunk/KDE/kdepim/plugins/kmail/bodypartformatter/text_xdiff.cpp #776104:776105
@@ -71,22 +71,22 @@
       const QString diff = bodyPart->asText();
       if ( diff.isEmpty() ) return AsIcon;
 
-      const QString addedLineStyle( "style=\"" "color: green;\"" );
-      const QLatin1String fileAddStyle( "style=\"font-weight: bold; " "color: green; \"" );
+      static const QLatin1String addedLineStyle( "style=\"" "color: green;\"" );
+      static const QLatin1String fileAddStyle( "style=\"font-weight: bold; " "color: green; \"" );
 
-      const QLatin1String removedLineStyle( "style=\"" "color: red;\"" );
-      const QLatin1String fileRemoveStyle( "style=\"font-weight: bold; "
+      static const QLatin1String removedLineStyle( "style=\"" "color: red;\"" );
+      static const QLatin1String fileRemoveStyle( "style=\"font-weight: bold; "
                                "color: red ;\"" );
 
-      const QLatin1String tableStyle(
+      static const QLatin1String tableStyle(
         "style=\""
         "text-align: -khtml-auto; "
         "border: solid black 1px; "
         "padding: 0.5em; "
         "margin: 0em;\"");
 
-      const QLatin1String sepStyle( "style=\"color: black; font-weight: bold;\"" );
-      const QLatin1String chunkStyle( "style=\"color: blue;\"" );
+      static const QLatin1String sepStyle( "style=\"color: black; font-weight: bold;\"" );
+      static const QLatin1String chunkStyle( "style=\"color: blue;\"" );
 
       QString html = "<br><div align=\"center\">";
       html += "<pre " + tableStyle + '>';
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic