Advertisement
Guest User

Untitled

a guest
Jul 30th, 2014
158
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 5.46 KB | None | 0 0
  1. commit 216fa3cc16df713c0f4a3f3dc29441eff3ab7a30
  2. Author: hjk <hjk121@nokiamail.com>
  3. Date: Wed Jul 30 15:10:56 2014 +0200
  4.  
  5. TextEditor: Update tab settings for all tab changes
  6.  
  7. Not just for selected documents. This also removes the need for
  8. the separate setupAsPlainTextDocument function.
  9.  
  10. Change-Id: I51a04eebbad37a7598068ce6e157a53c1519464f
  11.  
  12. diff --git a/src/plugins/android/androidmanifestdocument.cpp b/src/plugins/android/androidmanifestdocument.cpp
  13. index 27a0c47..0abb250 100644
  14. --- a/src/plugins/android/androidmanifestdocument.cpp
  15. +++ b/src/plugins/android/androidmanifestdocument.cpp
  16. @@ -41,7 +41,6 @@ using namespace Android::Internal;
  17. AndroidManifestDocument::AndroidManifestDocument(AndroidManifestEditorWidget *editorWidget)
  18. : m_editorWidget(editorWidget)
  19. {
  20. - setupAsPlainTextDocument();
  21. setId(Constants::ANDROID_MANIFEST_EDITOR_ID);
  22. setMimeType(QLatin1String(Constants::ANDROID_MANIFEST_MIME_TYPE));
  23. connect(editorWidget, SIGNAL(guiChanged()),
  24. diff --git a/src/plugins/cpptools/cpppointerdeclarationformatter_test.cpp b/src/plugins/cpptools/cpppointerdeclarationformatter_test.cpp
  25. index ffaa7c6..0361988 100644
  26. --- a/src/plugins/cpptools/cpppointerdeclarationformatter_test.cpp
  27. +++ b/src/plugins/cpptools/cpppointerdeclarationformatter_test.cpp
  28. @@ -103,7 +103,6 @@ public:
  29.  
  30. // Open file
  31. auto textDocument = new TextEditor::BaseTextDocument;
  32. - textDocument->setupAsPlainTextDocument();
  33. textDocument->setId(Core::Constants::K_DEFAULT_TEXT_EDITOR_ID);
  34. TextEditor::BaseTextEditorWidget editorWidget(textDocument);
  35. editorWidget.setupAsPlainEditor();
  36. diff --git a/src/plugins/designer/formwindowfile.cpp b/src/plugins/designer/formwindowfile.cpp
  37. index ae28fbc..24d2da9 100644
  38. --- a/src/plugins/designer/formwindowfile.cpp
  39. +++ b/src/plugins/designer/formwindowfile.cpp
  40. @@ -56,7 +56,6 @@ FormWindowFile::FormWindowFile(QDesignerFormWindowInterface *form, QObject *pare
  41. m_formWindow(form),
  42. m_isModified(false)
  43. {
  44. - setupAsPlainTextDocument();
  45. setMimeType(QLatin1String(Designer::Constants::FORM_MIMETYPE));
  46. setParent(parent);
  47. setId(Core::Id(Designer::Constants::K_DESIGNER_XML_EDITOR_ID));
  48. diff --git a/src/plugins/qnx/bardescriptoreditorwidget.cpp b/src/plugins/qnx/bardescriptoreditorwidget.cpp
  49. index 2692f29..f1edfdf 100644
  50. --- a/src/plugins/qnx/bardescriptoreditorwidget.cpp
  51. +++ b/src/plugins/qnx/bardescriptoreditorwidget.cpp
  52. @@ -169,7 +169,6 @@ void BarDescriptorEditorWidget::initAssetsPage()
  53. void BarDescriptorEditorWidget::initSourcePage()
  54. {
  55. auto doc = new TextEditor::BaseTextDocument;
  56. - doc->setupAsPlainTextDocument();
  57. doc->setId(Core::Constants::K_DEFAULT_TEXT_EDITOR_ID); // FIXME: This looks odd.
  58. doc->setIndenter(new TextEditor::NormalIndenter);
  59.  
  60. diff --git a/src/plugins/texteditor/basetextdocument.cpp b/src/plugins/texteditor/basetextdocument.cpp
  61. index adf5121..48d70f0 100644
  62. --- a/src/plugins/texteditor/basetextdocument.cpp
  63. +++ b/src/plugins/texteditor/basetextdocument.cpp
  64. @@ -82,7 +82,6 @@ public:
  65.  
  66. public slots:
  67. void onModificationChanged(bool modified);
  68. - void updateTabSettings();
  69.  
  70. public:
  71. QString m_defaultPath;
  72. @@ -200,13 +199,6 @@ void BaseTextDocumentPrivate::onModificationChanged(bool modified)
  73. updateRevisions();
  74. }
  75.  
  76. -void BaseTextDocumentPrivate::updateTabSettings()
  77. -{
  78. - if (Highlighter *highlighter = qobject_cast<Highlighter *>(m_highlighter))
  79. - highlighter->setTabSettings(m_tabSettings);
  80. -}
  81. -
  82. -
  83. ///////////////////////////////////////////////////////////////////////
  84. //
  85. // BaseTextDocument
  86. @@ -304,6 +296,10 @@ void BaseTextDocument::setTabSettings(const TextEditor::TabSettings &tabSettings
  87. if (tabSettings == d->m_tabSettings)
  88. return;
  89. d->m_tabSettings = tabSettings;
  90. +
  91. + if (Highlighter *highlighter = qobject_cast<Highlighter *>(d->m_highlighter))
  92. + highlighter->setTabSettings(tabSettings);
  93. +
  94. emit tabSettingsChanged();
  95. }
  96.  
  97. @@ -327,11 +323,6 @@ void BaseTextDocument::triggerPendingUpdates()
  98. applyFontSettings();
  99. }
  100.  
  101. -void BaseTextDocument::setupAsPlainTextDocument()
  102. -{
  103. - connect(this, SIGNAL(tabSettingsChanged()), d, SLOT(updateTabSettings()));
  104. -}
  105. -
  106. void BaseTextDocument::applyFontSettings()
  107. {
  108. d->m_fontSettingsNeedsApply = false;
  109. diff --git a/src/plugins/texteditor/basetextdocument.h b/src/plugins/texteditor/basetextdocument.h
  110. index 0384915..437c843 100644
  111. --- a/src/plugins/texteditor/basetextdocument.h
  112. +++ b/src/plugins/texteditor/basetextdocument.h
  113. @@ -128,8 +128,6 @@ public:
  114.  
  115. virtual void triggerPendingUpdates();
  116.  
  117. - void setupAsPlainTextDocument();
  118. -
  119. public slots:
  120. void setTabSettings(const TextEditor::TabSettings &tabSettings);
  121. void setFontSettings(const TextEditor::FontSettings &fontSettings);
  122. diff --git a/src/plugins/texteditor/plaintexteditorfactory.cpp b/src/plugins/texteditor/plaintexteditorfactory.cpp
  123. index 66b9892..2bcbc2a 100644
  124. --- a/src/plugins/texteditor/plaintexteditorfactory.cpp
  125. +++ b/src/plugins/texteditor/plaintexteditorfactory.cpp
  126. @@ -62,7 +62,6 @@ PlainTextEditorFactory::PlainTextEditorFactory(QObject *parent)
  127. Core::IEditor *PlainTextEditorFactory::createEditor()
  128. {
  129. auto doc = new BaseTextDocument;
  130. - doc->setupAsPlainTextDocument();
  131. doc->setId(Core::Constants::K_DEFAULT_TEXT_EDITOR_ID);
  132. doc->setIndenter(new NormalIndenter);
  133. auto widget = new BaseTextEditorWidget(doc);
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement