include cleanups (using iwyu)
authorMax Kellermann <max@musicpd.org>
Sun, 8 Sep 2019 18:39:37 +0000 (20:39 +0200)
committerMax Kellermann <max@musicpd.org>
Sun, 8 Sep 2019 18:54:49 +0000 (20:54 +0200)
47 files changed:
src/AsioServiceFwd.hxx
src/BasicColors.cxx
src/BasicMarquee.cxx
src/BasicMarquee.hxx
src/Bindings.hxx
src/ChatPage.cxx
src/FileBrowserPage.cxx
src/GlobalBindings.cxx
src/Instance.cxx
src/KeyName.cxx
src/LibraryPage.cxx
src/ListWindow.cxx
src/LyricsPage.cxx
src/Main.cxx
src/Match.cxx
src/Match.hxx
src/ProgressBar.cxx
src/ProxyPage.hxx
src/QueuePage.cxx
src/SearchPage.cxx
src/SongRowPaint.cxx
src/StatusBar.cxx
src/StatusBar.hxx
src/Styles.cxx
src/TextListRenderer.cxx
src/TextListRenderer.hxx
src/callbacks.hxx
src/conf.cxx
src/gidle.cxx
src/hscroll.cxx
src/i18n.h
src/lirc.cxx
src/lyrics.cxx
src/mpdclient.cxx
src/mpdclient.hxx
src/plugin.cxx
src/save_playlist.cxx
src/screen.cxx
src/screen.hxx
src/screen_find.cxx
src/screen_init.cxx
src/screen_keydef.cxx
src/screen_list.cxx
src/screen_paint.cxx
src/screen_utils.cxx
src/screen_utils.hxx
src/wreadln.cxx

index 7a00596..e6c60b2 100644 (file)
@@ -33,7 +33,7 @@
    boost::asio::io_context; eventually, we'll switch to the new API,
    but this would require dropping support for older Boost versions */
 
-#include <boost/asio/io_service.hpp>
+#include <boost/asio/io_service.hpp> // IWYU pragma: export
 
 #else
 namespace boost { namespace asio { class io_service; }}
index bba7103..f61cde9 100644 (file)
@@ -21,7 +21,7 @@
 
 #include <curses.h>
 
-#include <string.h>
+#include <strings.h>
 #include <stdlib.h>
 
 static constexpr const char *basic_color_names[] = {
index eb891f1..feab372 100644 (file)
@@ -20,8 +20,6 @@
 #include "BasicMarquee.hxx"
 #include "util/LocaleString.hxx"
 
-#include <algorithm>
-
 #include <assert.h>
 #include <string.h>
 
index f4dbb18..2ea333e 100644 (file)
@@ -23,6 +23,7 @@
 #include "util/Compiler.h"
 
 #include <string>
+#include <utility>
 
 /**
  * This class is used to auto-scroll text which does not fit on the
index 885f475..a8d5f5b 100644 (file)
@@ -20,7 +20,7 @@
 #ifndef BINDINGS_HXX
 #define BINDINGS_HXX
 
-#include "config.h"
+#include "config.h" // IWYU pragma: keep
 #include "Command.hxx"
 #include "util/Compiler.h"
 
index 6b3f059..17b6365 100644 (file)
@@ -25,6 +25,7 @@
 #include "mpdclient.hxx"
 #include "i18n.h"
 #include "charset.hxx"
+#include "Command.hxx"
 #include "Options.hxx"
 
 #include <mpd/idle.h>
index 4df4408..b457956 100644 (file)
 #include "screen_status.hxx"
 #include "save_playlist.hxx"
 #include "screen.hxx"
-#include "config.h"
+#include "config.h" // IWYU pragma: keep
 #include "i18n.h"
 #include "charset.hxx"
 #include "mpdclient.hxx"
 #include "filelist.hxx"
 #include "screen_utils.hxx"
 #include "screen_client.hxx"
+#include "Command.hxx"
 #include "Options.hxx"
 #include "util/UriUtil.hxx"
 
index 4c82c9b..4b9b704 100644 (file)
@@ -19,6 +19,7 @@
 
 #include "GlobalBindings.hxx"
 #include "Bindings.hxx"
+#include "config.h"
 
 #include <curses.h>
 
index 5c8688e..a2820a2 100644 (file)
@@ -19,7 +19,8 @@
 
 #include "Instance.hxx"
 #include "Options.hxx"
-#include "xterm_title.hxx"
+
+#include <signal.h>
 
 Instance::Instance()
        :io_service(),
index ce0135d..a270d3f 100644 (file)
@@ -22,6 +22,8 @@
 
 #include <curses.h>
 
+#include <stdio.h>
+
 const char *
 key2str(int key)
 {
index 7d60163..0bd5419 100644 (file)
 #include "LibraryPage.hxx"
 #include "TagListPage.hxx"
 #include "PageMeta.hxx"
-#include "screen_status.hxx"
-#include "screen_find.hxx"
 #include "FileListPage.hxx"
 #include "Command.hxx"
-#include "screen.hxx"
 #include "ProxyPage.hxx"
 #include "i18n.h"
 #include "charset.hxx"
@@ -32,9 +29,9 @@
 #include "filelist.hxx"
 #include "Options.hxx"
 
-#include <vector>
+#include <list>
 #include <string>
-#include <algorithm>
+#include <vector>
 
 #include <assert.h>
 #include <string.h>
index 2e481e8..9f32711 100644 (file)
@@ -22,6 +22,7 @@
 #include "ListText.hxx"
 #include "config.h"
 #include "Match.hxx"
+#include "Options.hxx"
 #include "Command.hxx"
 #include "paint.hxx"
 #include "screen_status.hxx"
@@ -29,9 +30,6 @@
 #include "i18n.h"
 
 #include <assert.h>
-#include <stdlib.h>
-#include <unistd.h>
-#include <string.h>
 
 void
 ListWindow::Paint(const ListRenderer &renderer) const noexcept
index 675c7c5..2fe3ef8 100644 (file)
@@ -23,6 +23,7 @@
 #include "FileBrowserPage.hxx"
 #include "SongPage.hxx"
 #include "i18n.h"
+#include "Command.hxx"
 #include "Options.hxx"
 #include "mpdclient.hxx"
 #include "screen.hxx"
index 69e2959..50ac21b 100644 (file)
@@ -33,9 +33,9 @@
 #include "xterm_title.hxx"
 #include "strfsong.hxx"
 #include "i18n.h"
-#include "player_command.hxx"
 #include "util/ScopeExit.hxx"
 #include "util/StringUTF8.hxx"
+#include "util/Compiler.h"
 
 #ifndef NCMPC_MINI
 #include "conf.hxx"
 
 #include <mpd/client.h>
 
-#include <stdlib.h>
-#include <unistd.h>
-#include <fcntl.h>
-#include <signal.h>
+#include <curses.h>
+
+#include <assert.h>
+#include <stdio.h>
 #include <string.h>
 
 #ifdef ENABLE_LOCALE
index a447a40..c2029b4 100644 (file)
@@ -18,7 +18,6 @@
  */
 
 #include "Match.hxx"
-#include "charset.hxx"
 
 #include <assert.h>
 #include <string.h>
index 4f4f780..ba4c4df 100644 (file)
@@ -25,9 +25,9 @@
 
 #ifdef HAVE_PCRE
 #include <pcre.h>
-#endif
-
+#else
 #include <stddef.h>
+#endif
 
 class MatchExpression {
 #ifndef HAVE_PCRE
index b1269b6..c818df6 100644 (file)
@@ -20,6 +20,7 @@
 #include "ProgressBar.hxx"
 #include "Styles.hxx"
 #include "Options.hxx"
+#include "config.h"
 
 #include <assert.h>
 
index 3a9d707..0eb68d3 100644 (file)
@@ -21,6 +21,7 @@
 #define NCMPC_PROXY_PAGE_HXX
 
 #include "Page.hxx"
+#include "config.h"
 
 class ProxyPage : public Page {
        WINDOW *const w;
index 35a39d5..c2de5a7 100644 (file)
@@ -29,6 +29,7 @@
 #include "config.h"
 #include "i18n.h"
 #include "charset.hxx"
+#include "Command.hxx"
 #include "Options.hxx"
 #include "mpdclient.hxx"
 #include "strfsong.hxx"
index e290546..e8d0367 100644 (file)
@@ -27,7 +27,6 @@
 #include "GlobalBindings.hxx"
 #include "charset.hxx"
 #include "mpdclient.hxx"
-#include "strfsong.hxx"
 #include "screen_utils.hxx"
 #include "FileListPage.hxx"
 #include "filelist.hxx"
index c959d16..6f62410 100644 (file)
@@ -22,7 +22,7 @@
 #include "strfsong.hxx"
 #include "time_format.hxx"
 #include "hscroll.hxx"
-#include "config.h"
+#include "config.h" // IWYU pragma: keep
 #include "util/LocaleString.hxx"
 
 #include <mpd/client.h>
index 88a9725..102d0b6 100644 (file)
@@ -28,7 +28,6 @@
 
 #include <mpd/client.h>
 
-#include <assert.h>
 #include <string.h>
 
 StatusBar::StatusBar(boost::asio::io_service &io_service,
index 85a3408..4827338 100644 (file)
@@ -20,7 +20,7 @@
 #ifndef NCMPC_STATUS_BAR_HXX
 #define NCMPC_STATUS_BAR_HXX
 
-#include "config.h"
+#include "config.h" // IWYU pragma: keep
 #include "AsioServiceFwd.hxx"
 #include "Window.hxx"
 
@@ -28,8 +28,6 @@
 #include "hscroll.hxx"
 #endif
 
-#include <mpd/status.h>
-
 #include <boost/asio/steady_timer.hpp>
 
 #include <string>
index 66e75c3..ea7236c 100644 (file)
@@ -22,6 +22,7 @@
 #include "CustomColors.hxx"
 #include "i18n.h"
 #include "util/StringStrip.hxx"
+#include "util/Compiler.h"
 
 #ifdef ENABLE_COLORS
 #include "Options.hxx"
@@ -29,7 +30,7 @@
 
 #include <assert.h>
 #include <stdio.h>
-#include <stdlib.h>
+#include <strings.h>
 #include <string.h>
 
 /**
index ad710fe..ef71343 100644 (file)
@@ -17,7 +17,6 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
-#include "config.h"
 #include "TextListRenderer.hxx"
 #include "ListText.hxx"
 #include "paint.hxx"
index c47d378..7de6580 100644 (file)
@@ -22,7 +22,6 @@
 
 #include "ListRenderer.hxx"
 
-class ScreenManager;
 class ListText;
 
 class TextListRenderer final : public ListRenderer {
index ea366bd..289f0c3 100644 (file)
@@ -20,8 +20,6 @@
 #ifndef NCMPC_CALLBACKS_H
 #define NCMPC_CALLBACKS_H
 
-#include <mpd/client.h>
-
 struct mpdclient;
 
 /**
index ef2689f..a4d3819 100644 (file)
 #include "Options.hxx"
 #include "io/Path.hxx"
 #include "util/CharUtil.hxx"
-#include "util/ScopeExit.hxx"
 #include "util/StringStrip.hxx"
 
 #include <assert.h>
 #include <sys/stat.h>
 #include <ctype.h>
 #include <stdio.h>
-#include <errno.h>
 #include <stdlib.h>
 #include <string.h>
 
index 0880d82..5df9eb1 100644 (file)
 */
 
 #include "gidle.hxx"
-#include "util/Compiler.h"
 
 #include <mpd/async.h>
 #include <mpd/parser.h>
 
 #include <assert.h>
 #include <string.h>
-#include <errno.h>
 
 MpdIdleSource::MpdIdleSource(boost::asio::io_service &io_service,
                             struct mpd_connection &_connection,
index 624115d..b3fd5c5 100644 (file)
@@ -19,9 +19,6 @@
 
 #include "hscroll.hxx"
 #include "Styles.hxx"
-#include "charset.hxx"
-
-#include <algorithm>
 
 #include <assert.h>
 
index 7bbc881..94e7330 100644 (file)
@@ -24,7 +24,7 @@
 
 #ifdef ENABLE_NLS
 
-#include <libintl.h>
+#include <libintl.h> // IWYU pragma: export
 
 #define _(x) gettext(x)
 
index 23ccd7b..db84d3d 100644 (file)
@@ -19,8 +19,8 @@
 
 #include "lirc.hxx"
 #include "ncmpc.hxx"
-#include "Bindings.hxx"
-#include "util/Compiler.h"
+#include "Command.hxx"
+#include "config.h"
 
 #include <lirc/lirc_client.h>
 
index d9c8ca1..c50f057 100644 (file)
@@ -20,8 +20,6 @@
 #include "lyrics.hxx"
 #include "config.h"
 
-#include <assert.h>
-
 static PluginList empty, plugins;
 
 void lyrics_init()
index 68e6c7d..a6dad42 100644 (file)
@@ -19,7 +19,6 @@
 
 #include "mpdclient.hxx"
 #include "callbacks.hxx"
-#include "filelist.hxx"
 #include "config.h"
 #include "gidle.hxx"
 #include "charset.hxx"
index 013b517..909e78d 100644 (file)
 #include "aconnect.hxx"
 #endif
 
-#include <mpd/client.h>
+#include <mpd/client.h> // IWYU pragma: export
 
 #include <boost/asio/steady_timer.hpp>
 
 #include <string>
 
 struct AsyncMpdConnect;
-struct MpdQueue;
-class MpdIdleSource;
-class FileList;
 
 struct mpdclient final
        : MpdIdleHandler
index 13dca5b..cf89962 100644 (file)
@@ -19,7 +19,6 @@
 
 #include "plugin.hxx"
 #include "io/Path.hxx"
-#include "util/Compiler.h"
 #include "util/ScopeExit.hxx"
 #include "util/UriUtil.hxx"
 
@@ -33,7 +32,6 @@
 #include <stdlib.h>
 #include <unistd.h>
 #include <dirent.h>
-#include <string.h>
 #include <signal.h>
 #include <sys/stat.h>
 #include <sys/wait.h>
index 404a4ea..113f0df 100644 (file)
 #include "save_playlist.hxx"
 #include "db_completion.hxx"
 #include "screen_status.hxx"
-#include "config.h"
+#include "config.h" // IWYU pragma: keep
 #include "i18n.h"
 #include "charset.hxx"
 #include "mpdclient.hxx"
-#include "wreadln.hxx"
 #include "Completion.hxx"
 #include "screen_utils.hxx"
 #include "util/Compiler.h"
 
 #include <mpd/client.h>
 
-#include <string.h>
+#include <stdio.h>
 
 #ifndef NCMPC_MINI
 
index 584c95f..573e276 100644 (file)
@@ -25,7 +25,6 @@
 #include "Command.hxx"
 #include "config.h"
 #include "i18n.h"
-#include "charset.hxx"
 #include "mpdclient.hxx"
 #include "Options.hxx"
 #include "DelayedSeek.hxx"
 
 #include <mpd/client.h>
 
-#include <stdlib.h>
-#include <unistd.h>
 #include <string.h>
-#include <time.h>
 
 ScreenManager::PageMap::iterator
 ScreenManager::MakePage(const PageMeta &sf)
index 14b3e0e..e47fe1d 100644 (file)
 
 #include <curses.h>
 
-#include <mpd/client.h>
-
+#include <map>
 #include <memory>
 #include <string>
-#include <map>
+#include <utility>
 
 enum class Command : unsigned;
+struct mpd_song;
 struct mpdclient;
 struct PageMeta;
 class Page;
index a176aca..8cc5258 100644 (file)
@@ -24,6 +24,7 @@
 #include "ListWindow.hxx"
 #include "AsyncUserInput.hxx"
 #include "i18n.h"
+#include "Command.hxx"
 #include "Options.hxx"
 #include "util/LocaleString.hxx"
 
index b5c0873..1d1ba6e 100644 (file)
 
 #include "screen.hxx"
 #include "Page.hxx"
-#include "screen_list.hxx"
 #include "QueuePage.hxx"
 #include "config.h"
-#include "i18n.h"
 #include "Options.hxx"
 #include "Styles.hxx"
 
-#include <stdlib.h>
-
 /* minimum window size */
 static const unsigned SCREEN_MIN_COLS = 14;
 static const unsigned SCREEN_MIN_ROWS = 5;
index aaf9a62..a072ae7 100644 (file)
@@ -25,7 +25,6 @@
 #include "ProxyPage.hxx"
 #include "screen_status.hxx"
 #include "screen_find.hxx"
-#include "screen.hxx"
 #include "KeyName.hxx"
 #include "i18n.h"
 #include "conf.hxx"
@@ -35,8 +34,6 @@
 #include "Options.hxx"
 #include "util/Compiler.h"
 
-#include <algorithm>
-
 #include <assert.h>
 #include <errno.h>
 #include <string.h>
index 5a1aeeb..897a33a 100644 (file)
@@ -19,7 +19,6 @@
 
 #include "screen_list.hxx"
 #include "PageMeta.hxx"
-#include "screen.hxx"
 #include "HelpPage.hxx"
 #include "QueuePage.hxx"
 #include "FileBrowserPage.hxx"
@@ -31,6 +30,7 @@
 #include "OutputsPage.hxx"
 #include "ChatPage.hxx"
 #include "util/Macros.hxx"
+#include "config.h"
 
 #include <string.h>
 
index 1a81dcb..ba442af 100644 (file)
 
 #include "screen.hxx"
 #include "Page.hxx"
-#include "config.h"
-#include "mpdclient.hxx"
 #include "Options.hxx"
 
-#include <mpd/client.h>
-
 #include <assert.h>
 
 void
index dfe4704..84aab59 100644 (file)
 
 #include "screen_utils.hxx"
 #include "screen.hxx"
-#include "mpdclient.hxx"
 #include "config.h"
 #include "i18n.h"
 #include "Options.hxx"
 #include "Styles.hxx"
 #include "wreadln.hxx"
 #include "ncmpc.hxx"
+#include "config.h"
 
 #ifndef _WIN32
 #include "WaitUserInput.hxx"
 #endif
 
-#include <mpd/client.h>
-
 #include <string.h>
 
 void
index 534d6d5..3b831ad 100644 (file)
 #ifndef SCREEN_UTILS_H
 #define SCREEN_UTILS_H
 
-#include "config.h"
-#include "Command.hxx"
 #include "History.hxx"
 #include "Completion.hxx"
 
-struct mpdclient;
-class Completion;
-
 /* sound an audible and/or visible bell */
 void
 screen_bell() noexcept;
index c645188..fc0757d 100644 (file)
@@ -19,7 +19,6 @@
 
 #include "wreadln.hxx"
 #include "Completion.hxx"
-#include "charset.hxx"
 #include "screen_utils.hxx"
 #include "Point.hxx"
 #include "config.h"