[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master 25b5804 3/6: Remove the now unnecessary wx/de
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master 25b5804 3/6: Remove the now unnecessary wx/defs.h inclusions |
Date: |
Mon, 13 Mar 2017 09:46:14 -0400 (EDT) |
branch: master
commit 25b580436e56e57c2cab67732370bd20d8cffbfa
Author: Vadim Zeitlin <address@hidden>
Commit: Gregory W. Chicares <address@hidden>
Remove the now unnecessary wx/defs.h inclusions
The lines including wx/defs.h were only needed in order to mark the file
as using wxWidgets before in order for the makefile heuristic to
recognize it as such and use the appropriately relaxed warning options
for it, but this is not necessary any longer after the last commit, so
these lines can now be simply removed.
---
database_document.cpp | 7 -------
rounding_document.cpp | 7 -------
tier_document.cpp | 7 -------
3 files changed, 21 deletions(-)
diff --git a/database_document.cpp b/database_document.cpp
index 21a1ffb..0ca2e6a 100644
--- a/database_document.cpp
+++ b/database_document.cpp
@@ -26,13 +26,6 @@
#include "alert.hpp"
#include "contains.hpp"
-// EVGENIY !! Doesn't it seem strange that this wx header appears
-// to be needed here? I don't see it included in similar files.
-// I tried omitting it, but wasn't able to figure out what the
-// diagnostics really meant.
-
-#include <wx/defs.h>
-
IMPLEMENT_DYNAMIC_CLASS(DatabaseDocument, ProductEditorDocument)
DatabaseDocument::DatabaseDocument()
diff --git a/rounding_document.cpp b/rounding_document.cpp
index d910afe..c7d2e64 100644
--- a/rounding_document.cpp
+++ b/rounding_document.cpp
@@ -30,13 +30,6 @@
#include "rounding_view_editor.hpp" // class RoundingButtons
#include "view_ex.tpp"
-// EVGENIY !! Doesn't it seem strange that this wx header appears
-// to be needed here? I don't see it included in similar files.
-// I tried omitting it, but wasn't able to figure out what the
-// diagnostics really meant.
-
-#include <wx/defs.h>
-
IMPLEMENT_DYNAMIC_CLASS(RoundingDocument, ProductEditorDocument)
RoundingDocument::RoundingDocument()
diff --git a/tier_document.cpp b/tier_document.cpp
index a568729..2c70feb 100644
--- a/tier_document.cpp
+++ b/tier_document.cpp
@@ -25,13 +25,6 @@
#include "ieee754.hpp" // infinity<>()
-// EVGENIY !! Doesn't it seem strange that this wx header appears
-// to be needed here? I don't see it included in similar files.
-// I tried omitting it, but wasn't able to figure out what the
-// diagnostics really meant.
-
-#include <wx/defs.h>
-
#include <vector>
IMPLEMENT_DYNAMIC_CLASS(TierDocument, ProductEditorDocument)
- [lmi-commits] [lmi] master updated (197119c -> b98ef0c), Greg Chicares, 2017/03/13
- [lmi-commits] [lmi] master 859e100 1/6: Move implementation of pure virtual dtor out of header, Greg Chicares, 2017/03/13
- [lmi-commits] [lmi] master 9ef5990 2/6: Disable gcc -Wcast-qual warning for wx in the code, not makefile, Greg Chicares, 2017/03/13
- [lmi-commits] [lmi] master b98ef0c 6/6: Refactor for clarity [392], Greg Chicares, 2017/03/13
- [lmi-commits] [lmi] master ba65912 5/6: Include the appropriate PCH header, Greg Chicares, 2017/03/13
- [lmi-commits] [lmi] master 351bf57 4/6: Improve documentation, Greg Chicares, 2017/03/13
- [lmi-commits] [lmi] master 25b5804 3/6: Remove the now unnecessary wx/defs.h inclusions,
Greg Chicares <=