Clean up

messagecomposer/src/sender/messagesender.h

@@ -63,8 +63,6 @@ public:
        message on success, so DO NOT DELETE OR MODIFY the message
        further.
 
-       FIXME: what about send() == false?
-
        @return true on success.
     */
     bool send(const KMime::Message::Ptr &msg, SendMethod method = SendDefault)

messagelist/src/utils/themeeditor.cpp

@@ -67,7 +67,6 @@ static const char gThemeContentItemTypeDndMimeDataFormat[] = "application/x-kmai
 ThemeColumnPropertiesDialog::ThemeColumnPropertiesDialog(QWidget *parent, Theme::Column *column, const QString &title)
     : QDialog(parent), mColumn(column)
 {
-    //setAttribute( Qt::WA_DeleteOnClose );
     setWindowModality(Qt::ApplicationModal);   // FIXME: Sure ?
     QVBoxLayout *mainLayout = new QVBoxLayout(this);
     QDialogButtonBox *buttonBox = new QDialogButtonBox(QDialogButtonBox::Ok | QDialogButtonBox::Cancel, this);

messageviewer/src/messagepartthemes/default/defaultrenderer.cpp

@@ -360,27 +360,7 @@ bool containsExternalReferences(const QString &str, const QString &extraHead)
     }
     return false;
 }
-#if 0 //FIXME
-QString processHtml(const QString &htmlSource, QString &extraHead)
-{
-    // Create a DOM Document from the HTML source
-    QWebPage page(nullptr);
-    page.settings()->setAttribute(QWebSettings::JavascriptEnabled, false);
-    page.settings()->setAttribute(QWebSettings::JavaEnabled, false);
-    page.settings()->setAttribute(QWebSettings::PluginsEnabled, false);
-
-    page.settings()->setAttribute(QWebSettings::AutoLoadImages, false);
-
-    QWebFrame *frame = page.mainFrame();
-    frame->setHtml(htmlSource);
 
-    const QWebElement body = frame->documentElement().findFirst(QStringLiteral("body"));
-    const QWebElement header = frame->documentElement().findFirst(QStringLiteral("head"));
-
-    extraHead = header.toInnerXml();
-    return body.toInnerXml();
-}
-#endif
 class CacheHtmlWriter : public MimeTreeParser::HtmlWriter
 {
 public:

messageviewer/src/viewer/csshelperbase.cpp

@@ -586,8 +586,6 @@ QString CSSHelperBase::screenCssDefinitions(const CSSHelperBase *helper, bool fi
                        "  padding: 0px ! important;\n"
                        "  font-size:0.8em ! important;\n"
                        "  border:1px solid %6 ! important;\n"
-                       // FIXME: InfoBackground crashes KHTML
-                       //"  background-color:InfoBackground ! important;\n"
                        "  background-color:%5 ! important;\n"
                        "}\n\n"