Code review for standard library header usage (#207)

DDSView/ddsview.cpp

@@ -9,16 +9,16 @@
 
 #define WIN32_LEAN_AND_MEAN
 #define NOMINMAX
-#include <windows.h>
-
-#include <assert.h>
-#include <stdio.h>
-#include <dxgiformat.h>
-#include <d3d11.h>
+#include <Windows.h>
 
 #include <algorithm>
+#include <cassert>
+#include <cstdio>
 
-#include <directxmath.h>
+#include <dxgiformat.h>
+#include <d3d11_1.h>
+
+#include <DirectXMath.h>
 
 #pragma warning(disable : 4619 4616 26812)
 

DirectXTex/DirectXTex.h

@@ -11,9 +11,10 @@
 
 #pragma once
 
-#include <algorithm>
+#include <cstddef>
 #include <cstdint>
 #include <functional>
+#include <utility>
 #include <vector>
 
 #if !defined(__d3d11_h__) && !defined(__d3d11_x_h__) && !defined(__d3d12_h__) && !defined(__d3d12_x_h__) && !defined(__XBOX_D3D12_X__)
@@ -32,11 +33,11 @@
 
 #include <OCIdl.h>
 
-#define DIRECTX_TEX_VERSION 191
-
 struct IWICImagingFactory;
 struct IWICMetadataQueryReader;
 
+#define DIRECTX_TEX_VERSION 191
+
 
 namespace DirectX
 {

DirectXTex/DirectXTexP.h

@@ -1,5 +1,5 @@
 //-------------------------------------------------------------------------------------
-// DirectXTexp.h
+// DirectXTexP.h
 //
 // DirectX Texture Library - Private header
 //
@@ -85,12 +85,12 @@
 #define NOHELP
 #pragma warning(pop)
 
+#include <Windows.h>
+
 #ifndef _WIN32_WINNT_WIN10
 #define _WIN32_WINNT_WIN10 0x0A00
 #endif
 
-#include <Windows.h>
-
 #ifdef _GAMING_XBOX_SCARLETT
 #include <d3d12_xs.h>
 #elif defined(_GAMING_XBOX)
@@ -107,25 +107,21 @@
 
 #define _XM_NO_XMVECTOR_OVERLOADS_
 
-#include <DirectXMath.h>
-#include <DirectXPackedVector.h>
-#include <assert.h>
-
-#include <malloc.h>
-#include <memory>
-
-#include <vector>
-
-#include <time.h>
-#include <stdlib.h>
-#include <search.h>
-
-#include <Ole2.h>
-
 #include "DirectXTex.h"
 
-#include <wincodec.h>
+#include <algorithm>
+#include <cassert>
+#include <cstdlib>
+#include <ctime>
+#include <cstring>
+#include <memory>
 
+#include <DirectXPackedVector.h>
+
+#include <malloc.h>
+
+#include <Ole2.h>
+#include <wincodec.h>
 #include <wrl\client.h>
 
 #include "scoped.h"

Texassemble/AnimatedGif.cpp

@@ -24,7 +24,9 @@
 #define NOHELP
 #pragma warning(pop)
 
+#include <cstddef>
 #include <memory>
+#include <utility>
 #include <vector>
 
 #include <wrl/client.h>

Texassemble/texassemble.cpp

@@ -20,13 +20,14 @@
 #define NOHELP
 #pragma warning(pop)
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <assert.h>
-
+#include <cassert>
+#include <cstddef>
+#include <cstdio>
+#include <cstdlib>
 #include <fstream>
 #include <memory>
 #include <list>
+#include <utility>
 #include <vector>
 
 #include <wrl/client.h>

Texconv/ExtendedBMP.cpp

@@ -24,6 +24,7 @@
 
 #include <Windows.h>
 
+#include <cstddef>
 #include <cstdint>
 #include <memory>
 

Texconv/PortablePixMap.cpp

@@ -21,6 +21,7 @@
 
 #include <Windows.h>
 
+#include <cstddef>
 #include <cstdint>
 #include <memory>
 

Texconv/texconv.cpp

@@ -19,10 +19,11 @@
 #define NOHELP
 #pragma warning(pop)
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <assert.h>
-
+#include <algorithm>
+#include <cassert>
+#include <cstddef>
+#include <cstdio>
+#include <cstdlib>
 #include <fstream>
 #include <memory>
 #include <list>

Texdiag/texdiag.cpp

@@ -20,10 +20,11 @@
 #define NOHELP
 #pragma warning(pop)
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <assert.h>
-
+#include <algorithm>
+#include <cassert>
+#include <cstddef>
+#include <cstdio>
+#include <cstdlib>
 #include <fstream>
 #include <memory>
 #include <list>