Modernize "C" includes in lm.

Jeroen Vermeulen · Jeroen Vermeulen · commit b3c0a29044dd · 2015-03-28T20:28:45.000+07:00
This is one of those little chores in managing a long-lived C++
project: standard C headers like stdio.h and math.h now have their own
place in the C++ standard as resp. cstdio, cmath, and so on.  In this
branch the #include names are updated for the lm/ subdirectory.

C++11 adds cstdint, but to support compilation with the previous
standard, that change is left for later.
diff --git a/lm/bhiksha.hh b/lm/bhiksha.hh
@@ -19,9 +19,8 @@
 #include "util/sorted_uniform.hh"
 
 #include <algorithm>
-
 #include <stdint.h>
-#include <assert.h>
+#include <cassert>
 
 namespace lm {
 namespace ngram {
diff --git a/lm/blank.hh b/lm/blank.hh
@@ -2,9 +2,8 @@
 #define LM_BLANK_H
 
 #include <limits>
-
 #include <stdint.h>
-#include <math.h>
+#include <cmath>
 
 namespace lm {
 namespace ngram {
diff --git a/lm/build_binary_main.cc b/lm/build_binary_main.cc
@@ -9,9 +9,8 @@
 #include <iostream>
 #include <iomanip>
 #include <limits>
-
-#include <math.h>
-#include <stdlib.h>
+#include <cmath>
+#include <cstdlib>
 
 #ifdef WIN32
 #include "util/getopt.hh"
diff --git a/lm/builder/interpolate.cc b/lm/builder/interpolate.cc
@@ -8,8 +8,8 @@
 #include "util/fixed_array.hh"
 #include "util/murmur_hash.hh"
 
-#include <assert.h>
-#include <math.h>
+#include <cassert>
+#include <cmath>
 
 namespace lm { namespace builder {
 namespace {
diff --git a/lm/builder/joint_order.hh b/lm/builder/joint_order.hh
@@ -9,7 +9,7 @@
 #include <iostream>
 #endif
 
-#include <string.h>
+#include <cstring>
 
 namespace lm { namespace builder {
 
diff --git a/lm/builder/ngram.hh b/lm/builder/ngram.hh
@@ -5,10 +5,9 @@
 #include "lm/word_index.hh"
 
 #include <cstddef>
-
-#include <assert.h>
+#include <cassert>
 #include <stdint.h>
-#include <string.h>
+#include <cstring>
 
 namespace lm {
 namespace builder {
diff --git a/lm/builder/print.cc b/lm/builder/print.cc
@@ -7,8 +7,7 @@
 #include "util/stream/timer.hh"
 
 #include <sstream>
-
-#include <string.h>
+#include <cstring>
 
 namespace lm { namespace builder {
 
diff --git a/lm/builder/print.hh b/lm/builder/print.hh
@@ -10,8 +10,7 @@
 #include "util/string_piece.hh"
 
 #include <ostream>
-
-#include <assert.h>
+#include <cassert>
 
 // Warning: print routines read all unigrams before all bigrams before all
 // trigrams etc.  So if other parts of the chain move jointly, you'll have to
diff --git a/lm/filter/arpa_io.cc b/lm/filter/arpa_io.cc
@@ -6,9 +6,9 @@
 #include <string>
 #include <vector>
 
-#include <ctype.h>
-#include <errno.h>
-#include <string.h>
+#include <cctype>
+#include <cerrno>
+#include <cstring>
 
 namespace lm {
 
diff --git a/lm/filter/arpa_io.hh b/lm/filter/arpa_io.hh
@@ -14,7 +14,7 @@
 #include <string>
 #include <vector>
 
-#include <string.h>
+#include <cstring>
 #include <stdint.h>
 
 namespace util { class FilePiece; }
diff --git a/lm/filter/phrase.cc b/lm/filter/phrase.cc
@@ -9,7 +9,7 @@
 #include <string>
 #include <vector>
 
-#include <ctype.h>
+#include <cctype>
 
 namespace lm {
 namespace phrase {
diff --git a/lm/filter/vocab.cc b/lm/filter/vocab.cc
@@ -3,7 +3,7 @@
 #include <istream>
 #include <iostream>
 
-#include <ctype.h>
+#include <cctype>
 
 namespace lm {
 namespace vocab {
diff --git a/lm/lm_exception.cc b/lm/lm_exception.cc
@@ -1,7 +1,7 @@
 #include "lm/lm_exception.hh"
 
-#include<errno.h>
-#include<stdio.h>
+#include <cerrno>
+#include <cstdio>
 
 namespace lm {
 
diff --git a/lm/model.hh b/lm/model.hh
@@ -17,8 +17,7 @@
 
 #include <algorithm>
 #include <vector>
-
-#include <string.h>
+#include <cstring>
 
 namespace util { class FilePiece; }
 
diff --git a/lm/model_test.cc b/lm/model_test.cc
@@ -1,7 +1,7 @@
 #include "lm/model.hh"
 
-#include <stdlib.h>
-#include <string.h>
+#include <cstdlib>
+#include <cstring>
 
 #define BOOST_TEST_MODULE ModelTest
 #include <boost/test/unit_test.hpp>
diff --git a/lm/ngram_query.hh b/lm/ngram_query.hh
@@ -11,8 +11,7 @@
 #include <ostream>
 #include <istream>
 #include <string>
-
-#include <math.h>
+#include <cmath>
 
 namespace lm {
 namespace ngram {
diff --git a/lm/partial.hh b/lm/partial.hh
@@ -5,8 +5,7 @@
 #include "lm/state.hh"
 
 #include <algorithm>
-
-#include <assert.h>
+#include <cassert>
 
 namespace lm {
 namespace ngram {
diff --git a/lm/read_arpa.cc b/lm/read_arpa.cc
@@ -9,8 +9,8 @@
 #include <sstream>
 #include <vector>
 
-#include <ctype.h>
-#include <string.h>
+#include <cctype>
+#include <cstring>
 #include <stdint.h>
 
 #ifdef WIN32
diff --git a/lm/search_trie.hh b/lm/search_trie.hh
@@ -12,8 +12,7 @@
 
 #include <vector>
 #include <cstdlib>
-
-#include <assert.h>
+#include <cassert>
 
 namespace lm {
 namespace ngram {
diff --git a/lm/state.hh b/lm/state.hh
@@ -5,7 +5,7 @@
 #include "lm/word_index.hh"
 #include "util/murmur_hash.hh"
 
-#include <string.h>
+#include <cstring>
 
 namespace lm {
 namespace ngram {
diff --git a/lm/trie.cc b/lm/trie.cc
@@ -5,7 +5,7 @@
 #include "util/exception.hh"
 #include "util/sorted_uniform.hh"
 
-#include <assert.h>
+#include <cassert>
 
 namespace lm {
 namespace ngram {
diff --git a/lm/virtual_interface.hh b/lm/virtual_interface.hh
@@ -6,7 +6,7 @@
 #include "util/string_piece.hh"
 
 #include <string>
-#include <string.h>
+#include <cstring>
 
 namespace lm {
 namespace base {
diff --git a/lm/vocab.cc b/lm/vocab.cc
@@ -12,8 +12,7 @@
 #include "util/probing_hash_table.hh"
 
 #include <string>
-
-#include <string.h>
+#include <cstring>
 
 namespace lm {
 namespace ngram {
diff --git a/lm/word_index.hh b/lm/word_index.hh
@@ -2,7 +2,7 @@
 #ifndef LM_WORD_INDEX_H
 #define LM_WORD_INDEX_H
 
-#include <limits.h>
+#include <climits>
 
 namespace lm {
 typedef unsigned int WordIndex;
diff --git a/lm/wrappers/nplm.cc b/lm/wrappers/nplm.cc
@@ -3,8 +3,7 @@
 #include "util/file.hh"
 
 #include <algorithm>
-
-#include <string.h>
+#include <cstring>
 
 #include "neuralLM.h"
 
