Can't remove remaining wx headers from other headers, comment that

lib-src/mod-nyq-bench/NyqBench.h

@@ -9,7 +9,7 @@
 #ifndef __NYQUIST_EFFECT_WORKBENCH__
 #define __NYQUIST_EFFECT_WORKBENCH__
 
-#include <wx/fdrepdlg.h> // wxFindReplaceData member variable below
+#include <wx/fdrepdlg.h> // wxFindReplaceData member variable
 #include <wx/string.h>
 
 #include <iostream>

src/AudacityApp.h

@@ -21,14 +21,14 @@
 
 #include "MemoryX.h"
 #include <wx/app.h> // to inherit
-#include <wx/dir.h>
-#include <wx/timer.h>
+#include <wx/dir.h> // for wxDIR_FILES
+#include <wx/timer.h> // member variable
 
 #include "widgets/FileHistory.h"
 #include "ondemand/ODTaskThread.h"
 
 #if defined(EXPERIMENTAL_CRASH_REPORT)
-#include <wx/debugrpt.h>
+#include <wx/debugrpt.h> // for wxDebugReport::Context
 #endif
 
 class wxSingleInstanceChecker;

src/AudioIO.h

@@ -23,8 +23,8 @@
 #include <atomic>
 #include <utility>
 #include <vector>
-#include <wx/atomic.h>
-#include <wx/weakref.h>
+#include <wx/atomic.h> // member variable
+#include <wx/weakref.h> // member variable
 
 #ifdef USE_MIDI
 

src/AutoRecovery.h

@@ -14,7 +14,7 @@
 #include "xml/XMLTagHandler.h"
 #include "xml/XMLWriter.h"
 
-#include <wx/mstream.h>
+#include <wx/mstream.h> // member variables
 
 #include <unordered_map>
 

src/BlockFile.h

@@ -19,7 +19,7 @@
 
 #include "SampleFormat.h"
 
-#include "wxFileNameWrapper.h" // member variable below
+#include "wxFileNameWrapper.h" // member variable
 
 #include "ondemand/ODTaskThread.h"
 

src/Dependencies.h

@@ -16,7 +16,7 @@
 
 #include <list>
 #include "MemoryX.h"
-#include "wxFileNameWrapper.h" // member variable below
+#include "wxFileNameWrapper.h" // member variable
 
 class AudacityProject;
 

src/DeviceChange.h

@@ -25,7 +25,7 @@
 
 #if defined(HAVE_DEVICE_CHANGE)
 
-#include <wx/timer.h>
+#include <wx/timer.h> // member variable
 
 class DeviceChangeInterface /* not final */
 {

src/FileException.h

@@ -10,7 +10,7 @@
 #define __AUDACITY_FILE_EXCEPTION__
 
 #include "AudacityException.h"
-#include <wx/filename.h> // wxFileName member variable below
+#include <wx/filename.h> // wxFileName member variable
 
 class FileException /* not final */ : public MessageBoxException
 {

src/MixerBoard.h

@@ -15,7 +15,7 @@
 #define __AUDACITY_MIXER_BOARD__
 
 #include <wx/frame.h> // to inherit
-#include <wx/scrolwin.h>
+#include <wx/scrolwin.h> // to inherit
 
 #include "widgets/ASlider.h"
 #include "widgets/wxPanelWrapper.h"

src/Prefs.h

@@ -33,7 +33,7 @@
 
 #include "../include/audacity/ComponentInterface.h"
 
-#include <wx/config.h>
+#include <wx/config.h> // to inherit wxFileConfig
 
 void InitPreferences();
 void FinishPreferences();

src/TimerRecordDialog.h

@@ -17,7 +17,7 @@
 #define __AUDACITY_TIMERRECORD_DIALOG__
 
 #include <wx/textctrl.h> // to inherit
-#include <wx/timer.h>
+#include <wx/timer.h> // member variable
 #include "export/Export.h"
 
 class wxCheckBox;

src/TrackPanel.h

@@ -18,7 +18,7 @@
 #include <vector>
 
 #include <wx/setup.h> // for wxUSE_* macros
-#include <wx/timer.h>
+#include <wx/timer.h> // to inherit
 
 #include "HitTestResult.h"
 

src/TrackPanelDrawingContext.h

@@ -17,7 +17,7 @@ class UIHandle;
 using UIHandlePtr = std::shared_ptr<UIHandle>;
 class wxDC;
 
-#include <wx/mousestate.h>
+#include <wx/mousestate.h> // member variable
 
 struct TrackPanelDrawingContext {
    wxDC &dc;

src/blockfile/ODDecodeBlockFile.h

@@ -32,7 +32,7 @@ Also, see ODPCMAliasBlockFile for a similar file.
 #include "../ondemand/ODTaskThread.h"
 #include "../DirManager.h"
 #include "../ondemand/ODDecodeTask.h"
-#include <wx/atomic.h>
+#include <wx/atomic.h> // member variable
 
 /// An AliasBlockFile that references uncompressed data in an existing file
 class ODDecodeBlockFile final : public SimpleBlockFile

src/effects/EffectRack.h

@@ -20,7 +20,7 @@
 
 #include <wx/defs.h>
 #include <wx/frame.h> // to inherit
-#include <wx/timer.h>
+#include <wx/timer.h> // member variable
 
 #include "EffectManager.h"
 

src/effects/VST/VSTControl.h

@@ -11,7 +11,7 @@
 #ifndef AUDACITY_VSTCONTROL_H
 #define AUDACITY_VSTCONTROL_H
 
-#include <wx/control.h>
+#include <wx/control.h> // to inherit
 
 #include "aeffectx.h"
 

src/effects/VST/VSTControlOSX.h

@@ -15,7 +15,7 @@
 #include <Carbon/Carbon.h>
 #endif
 
-#include <wx/osx/private.h>
+#include <wx/osx/private.h> // to inherit wxWidgetCocoaImpl
 
 #include "VSTControl.h"
 

src/effects/audiounits/AUControl.h

@@ -22,8 +22,8 @@
 #include <Carbon/Carbon.h>
 #endif
 
-#include <wx/osx/private.h>
-#include <wx/control.h>
+#include <wx/osx/private.h> // to inherit wxWidgetCocoaImpl
+#include <wx/control.h> // to inherit
 
 #include <AudioUnit/AudioComponent.h>
 #include <AudioUnit/AudioUnit.h>

src/export/ExportMultiple.h

@@ -13,7 +13,7 @@
 
 #include "Export.h"
 #include "../Tags.h"       // we need to know about the Tags class for metadata
-#include "../wxFileNameWrapper.h" // member variable below
+#include "../wxFileNameWrapper.h" // member variable
 
 class wxButton;
 class wxCheckBox;

src/prefs/ExtImportPrefs.h

@@ -12,7 +12,7 @@
 #define __AUDACITY_EXT_IMPORT_PREFS__
 
 #include <wx/defs.h>
-#include <wx/dnd.h>
+#include <wx/dnd.h> // to inherit wxDropTarget
 #include "../widgets/Grid.h"
 
 #include "PrefsPanel.h"

src/prefs/KeyConfigPrefs.h

@@ -16,7 +16,7 @@ class CommandManager;
 class ShuttleGui;
 
 #include <wx/defs.h>
-#include <wx/timer.h>
+#include <wx/timer.h> // member variable
 
 #include "../widgets/KeyView.h"
 

src/toolbars/ToolManager.h

@@ -15,9 +15,9 @@
 
 #include "../MemoryX.h"
 #include <wx/defs.h>
-#include <wx/eventfilter.h>
+#include <wx/eventfilter.h> // to inherit
 #include <wx/frame.h> // to inherit
-#include <wx/timer.h>
+#include <wx/timer.h> // member variable
 
 #include "ToolDock.h"
 #include "ToolBar.h"

src/widgets/ASlider.h

@@ -16,7 +16,7 @@
 #include "../MemoryX.h"
 #include <wx/setup.h> // for wxUSE_* macros
 #include <wx/defs.h>
-#include <wx/timer.h>
+#include <wx/timer.h> // member variable
 #include "widgets/wxPanelWrapper.h"
 
 #if wxUSE_ACCESSIBILITY

src/widgets/ErrorDialog.h

@@ -15,7 +15,7 @@
 #include "../Audacity.h"
 
 #include <wx/defs.h>
-#include <wx/msgdlg.h>
+#include <wx/msgdlg.h> // to inherit
 #include "LinkingHtmlWindow.h"
 #include "wxPanelWrapper.h"
 
@@ -99,7 +99,7 @@ inline int AudacityMessageBox(const wxString& message,
 }
 
 
-#include <wx/textdlg.h>
+#include <wx/textdlg.h> // to inherit
 
 /**************************************************************************//**
 \class AudacityTextEntryDialog

src/widgets/ExpandingToolBar.h

@@ -15,8 +15,8 @@
 #include <vector>
 #include <wx/defs.h>
 #include <wx/dragimag.h> // use macros and typedefs in this header
-#include <wx/timer.h>
-#include <wx/minifram.h>
+#include <wx/timer.h> // member variable
+#include <wx/minifram.h> // to inherit
 
 #include "ImageRoll.h"
 #include "wxPanelWrapper.h"

src/widgets/FileHistory.h

@@ -14,7 +14,7 @@
 #include <vector>
 #include <algorithm>
 #include <wx/defs.h>
-#include <wx/weakref.h>
+#include <wx/weakref.h> // member variable
 
 #include "audacity/Types.h"
 #include "../MemoryX.h"

src/widgets/KeyView.h

@@ -15,7 +15,7 @@
 #include <vector>
 #include <wx/setup.h> // for wxUSE_* macros
 #include <wx/defs.h>
-#include <wx/vlbox.h>
+#include <wx/vlbox.h> // to inherit wxVListBox
 
 #include "../commands/Keyboard.h"
 

src/widgets/Meter.h

@@ -19,7 +19,7 @@
 #include <wx/setup.h> // for wxUSE_* macros
 #include <wx/brush.h> // member variable
 #include <wx/defs.h>
-#include <wx/timer.h>
+#include <wx/timer.h> // member variable
 
 #include "../SampleFormat.h"
 #include "Ruler.h"

src/widgets/MultiDialog.h

@@ -16,7 +16,7 @@
 
 #include <wx/defs.h>
 
-#include <wx/chartype.h>
+#include <wx/chartype.h> // for typedef wxChar
 class wxString;
 
 const wxString &DefaultMultiDialogMessage();

src/widgets/ProgressDialog.h

@@ -23,7 +23,7 @@
 #include "../MemoryX.h"
 #include <vector>
 #include <wx/defs.h>
-#include <wx/evtloop.h>
+#include <wx/evtloop.h> // member variable
 
 #include "wxPanelWrapper.h"
 

src/widgets/numformatter.h

@@ -15,7 +15,7 @@
 #define _WIDGETS_NUMFORMATTER_H_
 
 #include <cstddef> // for size_t
-#include <wx/chartype.h>
+#include <wx/chartype.h> // for typedef wxChar
 #include <wx/longlong.h>
 
 class wxString;

src/widgets/valnum.h

@@ -18,7 +18,7 @@
 #if wxUSE_VALIDATORS
 
 #include <wx/textctrl.h> // complete type needed in template function
-#include <wx/validate.h>
+#include <wx/validate.h> // to inherit
 
 #include <limits>
 

src/widgets/wxPanelWrapper.h

@@ -105,7 +105,7 @@ public:
    }
 };
 
-#include <wx/dirdlg.h>
+#include <wx/dirdlg.h> // to inherit
 
 class AUDACITY_DLL_API wxDirDialogWrapper : public wxTabTraversalWrapper<wxDirDialog>
 {