shiboken: Small code cleanup

Fix some left-over nullptr issues, C-style casts and some signedness
issues.

Remove some unused functions and parameters. Remove empty statements.

Add defined() around macros, fixing warnings like:
warning: '_WINDOWS' is not defined, evaluates to 0

Change-Id: Idaa4334cb0edb3661ca766c248ec18ab85b0ee8e
Reviewed-by: Cristian Maureira-Fredes <cristian.maureira-fredes@qt.io>

1 files changed, 1 insertions, 1 deletions

diff --git a/sources/shiboken2/generator/shiboken2/overloaddata.cpp b/sources/shiboken2/generator/shiboken2/overloaddata.cpp
index 56b64bbd5..bd39e9444 100644
--- a/sources/shiboken2/generator/shiboken2/overloaddata.cpp
+++ b/sources/shiboken2/generator/shiboken2/overloaddata.cpp
@@ -106,7 +106,7 @@ struct OverloadSortData
             return;
         map[typeName] = counter;
         if (!reverseMap.contains(counter))
-            reverseMap[counter] = 0;
+            reverseMap[counter] = nullptr;
         counter++;
     }