Minor C++ changes noticed while normalizing with 1.12 (#1606)

c++/src/H5Attribute.cpp

@@ -11,6 +11,7 @@
  * help@hdfgroup.org.                                                        *
  * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
 
+#include <cstdlib>
 #include <iostream>
 #include <string>
 

c++/src/H5DataSet.cpp

@@ -11,6 +11,7 @@
  * help@hdfgroup.org.                                                        *
  * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
 
+#include <cstdlib>
 #include <iostream>
 #include <string>
 

c++/src/H5DataType.cpp

@@ -11,6 +11,7 @@
  * help@hdfgroup.org.                                                        *
  * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
 
+#include <cstdlib>
 #include <iostream>
 #include <string>
 

c++/src/H5FaccProp.cpp

@@ -17,8 +17,6 @@
 using std::cerr;
 using std::endl;
 
-//#include <string>
-
 #include "H5Include.h"
 #include "H5Exception.h"
 #include "H5IdComponent.h"

c++/src/H5Library.cpp

@@ -11,8 +11,8 @@
  * help@hdfgroup.org.                                                        *
  * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
 
-#include <string>
 #include <cstdlib>
+#include <string>
 
 #include "H5CppDoc.h" // included only for Doxygen to generate part of RM
 #include "H5Include.h"

c++/src/H5Location.cpp

@@ -11,8 +11,8 @@
  * help@hdfgroup.org.                                                        *
  * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
 
-#include <string>
 #include <iostream>
+#include <string>
 using namespace std;
 
 #include "H5Include.h"

c++/src/H5PropList.cpp

@@ -12,7 +12,6 @@
  * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
 
 #include <iostream>
-
 #include <string>
 
 #include "H5Include.h"

c++/test/dsets.cpp

@@ -412,7 +412,6 @@ const H5Z_class2_t H5Z_BOGUS[1] = {{
 static size_t
 filter_bogus(unsigned int flags, size_t cd_nelmts, const unsigned int cd_values[], size_t nbytes,
              size_t *buf_size, void **buf)
-// H5_ATTR_UNUSED variables caused warning, but taking them out caused failure.
 {
     // Unused
     (void)flags;