common: Remove redundant includes

Fixes: http://tracker.ceph.com/issues/19883 (Partially)
Signed-off-by: Brad Hubbard <bhubbard@redhat.com>

diff --git a/src/common/ConfUtils.cc b/src/common/ConfUtils.cc
index b8e8edbce022ea323bb8cc5cab79275d00c96775..135e7da17abf07a5256fab695073137b396ddd5b 100644 (file)
--- a/src/common/ConfUtils.cc
+++ b/src/common/ConfUtils.cc
@@ -14,7 +14,6 @@
 
 #include <algorithm>
 #include <errno.h>
-#include <list>
 #include <map>
 #include <sstream>
 #include <stdio.h>
@@ -22,8 +21,6 @@
 #include <string.h>
 #include <string>
 #include <sys/stat.h>
-#include <sys/types.h>
-#include <unistd.h>
 #include <iostream>
 
 #include "include/buffer.h"

diff --git a/src/common/Formatter.cc b/src/common/Formatter.cc
index 0bbbdd361ec95db938a669037b6a98947706025f..00a9b3849ca3ea125bfbecfe333e18714a622c7e 100644 (file)
--- a/src/common/Formatter.cc
+++ b/src/common/Formatter.cc
@@ -14,8 +14,6 @@
 
 #define LARGE_SIZE 1024
 
-#include "include/int_types.h"
-
 #include "assert.h"
 #include "Formatter.h"
 #include "HTMLFormatter.h"
@@ -23,11 +21,9 @@
 #include "include/buffer.h"
 
 #include <algorithm>
-#include <iostream>
 #include <sstream>
 #include <stdarg.h>
 #include <stdio.h>
-#include <stdlib.h>
 #include <vector>
 #include <string>
 #include <set>

diff --git a/src/common/Formatter.h b/src/common/Formatter.h
index a4cf3e13cd58c1040fdf417a733b3944b9082060..a23266ac4adf75386613ca997d158acafa964cb7 100644 (file)
--- a/src/common/Formatter.h
+++ b/src/common/Formatter.h
@@ -9,8 +9,10 @@
 #include <iosfwd>
 #include <list>
 #include <vector>
-#include <sstream>
 #include <stdarg.h>
+#include <stddef.h>
+#include <utility>
+#include <sstream>
 #include <string>
 #include <map>
 

diff --git a/src/common/HTMLFormatter.cc b/src/common/HTMLFormatter.cc
index 63474adbc1a321a2def1130fbc40f4fa74c07bfb..11004cc4879e0df411907c6c780326611aef910a 100644 (file)
--- a/src/common/HTMLFormatter.cc
+++ b/src/common/HTMLFormatter.cc
@@ -14,22 +14,15 @@
 
 #define LARGE_SIZE 1024
 
-#include "include/int_types.h"
-
-#include "assert.h"
 #include "Formatter.h"
 #include "HTMLFormatter.h"
-#include "common/escape.h"
 
-#include <iostream>
 #include <sstream>
 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
-#include <vector>
 #include <string>
-#include <set>
-#include <boost/format.hpp>
+#include <string.h>     // for strdup
 
 // -----------------------
 namespace ceph {

diff --git a/src/common/HTMLFormatter.h b/src/common/HTMLFormatter.h
index 23fc8765599ae4760c06c671339397e732e969c3..19d056f76af8825f45589563d417bb61cebcc97b 100644 (file)
--- a/src/common/HTMLFormatter.h
+++ b/src/common/HTMLFormatter.h
@@ -3,18 +3,10 @@
 #ifndef CEPH_HTML_FORMATTER_H
 #define CEPH_HTML_FORMATTER_H
 
-#include "include/int_types.h"
-
-#include <deque>
 #include <iosfwd>
-#include <list>
-#include <vector>
-#include <sstream>
 #include <stdarg.h>
 #include <string>
-#include <map>
 
-#include "include/buffer.h"
 #include "Formatter.h"
 
 namespace ceph {

diff --git a/src/common/SloppyCRCMap.cc b/src/common/SloppyCRCMap.cc
index 7924ae6e8a7e69a76283fac175c92ff5d48d181b..c637f7a257ac292d21a3cac97160963093118830 100644 (file)
--- a/src/common/SloppyCRCMap.cc
+++ b/src/common/SloppyCRCMap.cc
@@ -2,7 +2,8 @@
 // vim: ts=8 sw=2 smarttab
 
 #include "common/SloppyCRCMap.h"
-#include "common/Formatter.h"
+
+using namespace std;
 
 void SloppyCRCMap::write(uint64_t offset, uint64_t len, const bufferlist& bl,
                         std::ostream *out)

diff --git a/src/common/SloppyCRCMap.h b/src/common/SloppyCRCMap.h
index 34642a325394d4853a55006b383dc41123fd8955..78c470c1887b24c723c72eeabba917f1984896b8 100644 (file)
--- a/src/common/SloppyCRCMap.h
+++ b/src/common/SloppyCRCMap.h
@@ -4,9 +4,10 @@
 #ifndef CEPH_COMMON_SLOPPYCRCMAP_H
 #define CEPH_COMMON_SLOPPYCRCMAP_H
 
-#include "include/types.h"
 #include "include/encoding.h"
+#include "common/Formatter.h"
 
+#include <list>
 #include <map>
 #include <ostream>
 
@@ -69,7 +70,7 @@ public:
   void encode(bufferlist& bl) const;
   void decode(bufferlist::iterator& bl);
   void dump(Formatter *f) const;
-  static void generate_test_instances(list<SloppyCRCMap*>& ls);
+  static void generate_test_instances(std::list<SloppyCRCMap*>& ls);
 };
 WRITE_CLASS_ENCODER(SloppyCRCMap)
 

diff --git a/src/common/blkdev.cc b/src/common/blkdev.cc
index 361a7738bd452ccf73e5679707bcfab98dd26c3f..11bd954103b926699ed6d74288f80de9fa20ece5 100644 (file)
--- a/src/common/blkdev.cc
+++ b/src/common/blkdev.cc
@@ -18,7 +18,6 @@
 #include <ctype.h>
 #include <dirent.h>
 #include <stdlib.h>
-#include "include/int_types.h"
 #include "include/uuid.h"
 
 #ifdef __linux__

diff --git a/src/common/ceph_fs.cc b/src/common/ceph_fs.cc
index 923e38b59d00434f21278c889dcca80a5d1e12b7..ce9873121c2cf4cb3ab44d88989b2db842681303 100644 (file)
--- a/src/common/ceph_fs.cc
+++ b/src/common/ceph_fs.cc
@@ -4,8 +4,6 @@
  *
  */
 
-#include <errno.h>
-
 /*
  * Some non-inline ceph helpers
  */

diff --git a/src/common/crc32c_intel_baseline.c b/src/common/crc32c_intel_baseline.c
index 390898171df6a553f312a6421bf96fd8ffa70c8e..2862f627246e3e639285dc220800e7e8865fdf41 100644 (file)
--- a/src/common/crc32c_intel_baseline.c
+++ b/src/common/crc32c_intel_baseline.c
@@ -36,8 +36,6 @@
 
 #include "include/int_types.h"
 
-#include <stdlib.h>
-
 #define MAX_ITER       8
 
 unsigned long crc32_table_iscsi_base[256] = {

diff --git a/src/common/mime.c b/src/common/mime.c
index 90c19a7684e6ffa72946185f32c4f2169c0a797e..fe45123ccc94c0f3f8e030a2d127d3662fedf1ca 100644 (file)
--- a/src/common/mime.c
+++ b/src/common/mime.c
@@ -15,7 +15,6 @@
 
 #include <errno.h>
 #include <stdio.h>
-#include <string.h>
 
 int mime_encode_as_qp(const char *input, char *output, int outlen)
 {

diff --git a/src/common/pipe.c b/src/common/pipe.c
index 4d22f2458dc3b69b24f2b633c96683218beb2980..913aa4fe89ab43960c7b11bdbd2a4afca187662e 100644 (file)
--- a/src/common/pipe.c
+++ b/src/common/pipe.c
@@ -18,6 +18,8 @@
 
 #include <errno.h>
 #include <fcntl.h>
+#include <limits.h>
+#include <stdint.h>
 #include <unistd.h>
 
 int pipe_cloexec(int pipefd[2])

diff --git a/src/common/run_cmd.cc b/src/common/run_cmd.cc
index 81e41712b9b38b5a713c3ad345a9bd65a08ed8c5..396676d7b45cd499b6f2be6511380d14bba160f4 100644 (file)
--- a/src/common/run_cmd.cc
+++ b/src/common/run_cmd.cc
@@ -18,7 +18,6 @@
 #include <sstream>
 #include <stdarg.h>
 #include <stdlib.h>
-#include <sys/types.h>
 #include <sys/wait.h>
 #include <unistd.h>
 #include <vector>

diff --git a/src/common/strtol.cc b/src/common/strtol.cc
index 863ef08cf46d9e291821850cbf836bbbe28ed773..a8ddafe70b52968216d119dbe3fe3a5cd1df5eee 100644 (file)
--- a/src/common/strtol.cc
+++ b/src/common/strtol.cc
@@ -17,6 +17,7 @@
 #include <cerrno>
 #include <climits>
 #include <cstdlib>
+#include <limits>
 #include <sstream>
 
 using std::ostringstream;

diff --git a/src/common/strtol.h b/src/common/strtol.h
index 2f4c4ce2756d88d9ef0eb8b08506e1e89b5b5c94..810273ebd23258e33ee40f3908919d1f938237b0 100644 (file)
--- a/src/common/strtol.h
+++ b/src/common/strtol.h
@@ -16,7 +16,6 @@
 #define CEPH_COMMON_STRTOL_H
 
 #include <string>
-#include <limits>
 extern "C" {
 #include <stdint.h>
 }

diff --git a/src/common/utf8.c b/src/common/utf8.c
index 3bc77c32cded5578e33528fedd0f48f6b1b45870..5a8592f1cc5e2afbea9ef52b5b49c8674ca01ed0 100644 (file)
--- a/src/common/utf8.c
+++ b/src/common/utf8.c
@@ -13,7 +13,6 @@
  */
 #include "common/utf8.h"
 
-#include <stdio.h>
 #include <string.h>
 
 static int high_bits_set(int c)

diff --git a/src/common/version.cc b/src/common/version.cc
index 0ca569e9c5691aa1ab748f176ad0e4c5181378c9..99c9d58ed86cce4795dc1f7018fbf7fc59a5a382 100644 (file)
--- a/src/common/version.cc
+++ b/src/common/version.cc
@@ -12,7 +12,6 @@
  *
  */
 
-#include "acconfig.h"
 #include "ceph_ver.h"
 #include "common/version.h"
 

diff --git a/src/common/xattr.c b/src/common/xattr.c
index caa31d5270456a2750aafbe9e566f0ceefaa53a8..080bebcaafd81497867ab6113f3aeeb09f11c656 100644 (file)
--- a/src/common/xattr.c
+++ b/src/common/xattr.c
@@ -9,7 +9,6 @@
  * Foundation.  See file COPYING.
  */
 
-#include "acconfig.h"
 #if defined(__FreeBSD__)
 #include <errno.h>
 #include <stdint.h>

diff --git a/src/include/crc32c.h b/src/include/crc32c.h
index a568edabe1914cbafd1a19594cd8ebcb8b8def60..35b6cafcc3a58326109a380ae93c004152ee3642 100644 (file)
--- a/src/include/crc32c.h
+++ b/src/include/crc32c.h
@@ -1,8 +1,7 @@
 #ifndef CEPH_CRC32C_H
 #define CEPH_CRC32C_H
 
-#include <inttypes.h>
-#include <string.h>
+#include <stdint.h>
 
 typedef uint32_t (*ceph_crc32c_func_t)(uint32_t crc, unsigned char const *data, unsigned length);
 

diff --git a/src/include/msgr.h b/src/include/msgr.h
index f60c9dfa4938a5b62e609eacb89ff2a0c50e4072..1953eb28b40b44d2515259049c86f36c632ec08b 100644 (file)
--- a/src/include/msgr.h
+++ b/src/include/msgr.h
@@ -5,6 +5,8 @@
 #include <sys/socket.h> // for struct sockaddr_storage
 #endif
 
+#include "include/int_types.h"
+
 /*
  * Data types for message passing layer used by Ceph.
  */

diff --git a/src/include/str_list.h b/src/include/str_list.h
index 7f4dce535368d1ff5fb1819e1e0c97a5037dc8e7..8ca07f3d31bb6bfa12494580ac261e28610e117a 100644 (file)
--- a/src/include/str_list.h
+++ b/src/include/str_list.h
@@ -3,7 +3,6 @@
 
 #include <list>
 #include <set>
-#include <sstream>
 #include <string>
 #include <vector>
 

diff --git a/src/test/common/test_sloppy_crc_map.cc b/src/test/common/test_sloppy_crc_map.cc
index 1cdb6e459ded0cb414185e3105c4cf039a01089d..3eb855130aabd1a6b0416c86877d04244b72066c 100644 (file)
--- a/src/test/common/test_sloppy_crc_map.cc
+++ b/src/test/common/test_sloppy_crc_map.cc
@@ -1,10 +1,14 @@
 // -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
 // vim: ts=8 sw=2 smarttab
 
+#include <iostream>
+
 #include "common/SloppyCRCMap.h"
 #include "common/Formatter.h"
 #include <gtest/gtest.h>
 
+using namespace std;
+
 void dump(const SloppyCRCMap& scm)
 {
   Formatter *f = Formatter::create("json-pretty");

diff --git a/src/test/formatter.cc b/src/test/formatter.cc
index 35ec1d652b296fc6f47b8a62297fc7b32c1a7915..da8cc937e71246ad74f45c8e8073ee970430e5a7 100644 (file)
--- a/src/test/formatter.cc
+++ b/src/test/formatter.cc
@@ -19,6 +19,7 @@
 #include <sstream>
 #include <string>
 
+using namespace ceph;
 using std::ostringstream;
 
 TEST(JsonFormatter, Simple1) {