Cleanup warnings

- Renamed namespace to match directory
- Removed redundant unsafe keyword usage
- removed redundant value initializations
- Mark fields readonly that can be
- Use C# naming conventions (e.g. `_showAnotherWindow` instead of `show_another_window`

Author: AristurtleDev

Examples/ExampleMonoGame/DrawVertDeclaration.cs

@@ -1,7 +1,7 @@
 using Hexa.NET.ImGui;
 using Microsoft.Xna.Framework.Graphics;
 
-namespace MonoGameExample;
+namespace ExampleMonoGame;
 
 public static class DrawVertDeclaration
 {

Examples/ExampleMonoGame/ImGuiRenderer.cs

@@ -6,18 +6,18 @@
 using Microsoft.Xna.Framework.Graphics;
 using Microsoft.Xna.Framework.Input;
 
-namespace MonoGameExample;
+namespace ExampleMonoGame;
 
 public class ImGuiRenderer
 {
     private const float WHEEL_DELTA = 120;
 
-    private Game _game;
+    private readonly Game _game;
 
     // Graphics
-    private GraphicsDevice _graphicsDevice;
+    private readonly GraphicsDevice _graphicsDevice;
     private BasicEffect _effect;
-    private RasterizerState _rasterizerState;
+    private readonly RasterizerState _rasterizerState;
 
     private byte[] _vertexData;
     private VertexBuffer _vertexBuffer;
@@ -28,13 +28,13 @@ public class ImGuiRenderer
     private int _indexBufferSize;
 
     // Textures
-    private Dictionary<ImTextureID, TextureInfo> _textures;
-    private int _nextTexId = 0;
+    private readonly Dictionary<ImTextureID, TextureInfo> _textures;
+    private int _nextTexId;
 
     // Input
     private int _scrollWheelValue;
     private int _horizontalScrollWheelValue;
-    private Keys[] _allKeys = Enum.GetValues<Keys>();
+    private readonly Keys[] _allKeys = Enum.GetValues<Keys>();
 
 
     public ImGuiRenderer(Game game)
@@ -62,7 +62,7 @@ public ImGuiRenderer(Game game)
         SetupBackendCapabilities();
     }
 
-    private unsafe void SetupBackendCapabilities()
+    private void SetupBackendCapabilities()
     {
         ImGuiIOPtr io = ImGui.GetIO();
         io.BackendFlags |= ImGuiBackendFlags.RendererHasTextures;
@@ -109,7 +109,7 @@ public virtual void UnbindTexture(ImTextureRef textureRef)
         }
     }
 
-    public virtual unsafe void UpdateTexture(ImTextureDataPtr textureData)
+    public virtual void UpdateTexture(ImTextureDataPtr textureData)
     {
         switch (textureData.Status)
         {
@@ -468,8 +468,8 @@ private unsafe void UpdateBuffers(ImDrawData* drawData)
             {
                 fixed (void* idxDstPtr = &_indexData[idxOffset * sizeof(ushort)])
                 {
-                    Buffer.MemoryCopy((void*)cmdList->VtxBuffer.Data, vtxDstPtr, _vertexData.Length, cmdList->VtxBuffer.Size * DrawVertDeclaration.Size);
-                    Buffer.MemoryCopy((void*)cmdList->IdxBuffer.Data, idxDstPtr, _indexData.Length, cmdList->IdxBuffer.Size * sizeof(ushort));
+                    Buffer.MemoryCopy(cmdList->VtxBuffer.Data, vtxDstPtr, _vertexData.Length, cmdList->VtxBuffer.Size * DrawVertDeclaration.Size);
+                    Buffer.MemoryCopy(cmdList->IdxBuffer.Data, idxDstPtr, _indexData.Length, cmdList->IdxBuffer.Size * sizeof(ushort));
                 }
             }
 

Examples/ExampleMonoGame/Program.cs

@@ -1,2 +1,4 @@
-using var game = new MonoGameExample.SampleGame();
+using ExampleMonoGame;
+
+using var game = new SampleGame();
 game.Run();

Examples/ExampleMonoGame/SampleGame.cs

@@ -5,7 +5,7 @@
 using Microsoft.Xna.Framework.Graphics;
 using Num = System.Numerics;
 
-namespace MonoGameExample;
+namespace ExampleMonoGame;
 
 /// <summary>
 /// Simple Monogame + ImGui example
@@ -53,7 +53,7 @@ protected override void LoadContent()
 
     protected override void Draw(GameTime gameTime)
     {
-        GraphicsDevice.Clear(new Color(clear_color.X, clear_color.Y, clear_color.Z));
+        GraphicsDevice.Clear(new Color(_clearColor.X, _clearColor.Y, _clearColor.Z));
 
         // Call BeforeLayout first to set things up
         _imGuiRenderer.BeforeLayout(gameTime);
@@ -68,29 +68,29 @@ protected override void Draw(GameTime gameTime)
     }
 
     // Direct port of the example at https://github.com/ocornut/imgui/blob/master/examples/example_sdl2_opengl2/main.cpp
-    private float f = 0.0f;
-
-    private bool show_test_window = false;
-    private bool show_another_window = false;
-    private Num.Vector3 clear_color = new Num.Vector3(114f / 255f, 144f / 255f, 154f / 255f);
-    private byte[] _textBuffer = new byte[100];
+    private float _floatValue;
+    
+    private bool _showTestWindow;
+    private bool _showAnotherWindow;
+    private Num.Vector3 _clearColor = new Num.Vector3(114f / 255f, 144f / 255f, 154f / 255f);
+    private readonly byte[] _textBuffer = new byte[100];
 
     protected virtual void ImGuiLayout()
     {
         // 1. Show a simple window
         // Tip: if we don't call ImGui.Begin()/ImGui.End() the widgets appears in a window automatically called "Debug"
         {
             ImGui.Text("Hello, world!"u8);
-            ImGui.SliderFloat("float"u8, ref f, 0.0f, 1.0f, string.Empty);
-            ImGui.ColorEdit3("clear color", ref clear_color);
+            ImGui.SliderFloat("float"u8, ref _floatValue, 0.0f, 1.0f, string.Empty);
+            ImGui.ColorEdit3("clear color", ref _clearColor);
             if (ImGui.Button("Test Window"u8))
             {
-                show_test_window = !show_test_window;
+                _showTestWindow = !_showTestWindow;
             }
 
             if (ImGui.Button("Another Window"u8))
             {
-                show_another_window = !show_another_window;
+                _showAnotherWindow = !_showAnotherWindow;
             }
 
             // Use StrBuilder for dynamic text instead of string.Format
@@ -117,19 +117,19 @@ protected virtual void ImGuiLayout()
         }
 
         // 2. Show another simple window, this time using an explicit Begin/End pair
-        if (show_another_window)
+        if (_showAnotherWindow)
         {
             ImGui.SetNextWindowSize(new Num.Vector2(200, 100), ImGuiCond.FirstUseEver);
-            ImGui.Begin("Another Window"u8, ref show_another_window);
+            ImGui.Begin("Another Window"u8, ref _showAnotherWindow);
             ImGui.Text("Hello"u8);
             ImGui.End();
         }
 
         // 3. Show the ImGui test window. Most of the sample code is in ImGui.ShowTestWindow()
-        if (show_test_window)
+        if (_showTestWindow)
         {
             ImGui.SetNextWindowPos(new Num.Vector2(650, 20), ImGuiCond.FirstUseEver);
-            ImGui.ShowDemoWindow(ref show_test_window);
+            ImGui.ShowDemoWindow(ref _showTestWindow);
         }
     }
 

Examples/ExampleMonoGame/TextureInfo.cs

@@ -1,6 +1,6 @@
 using Microsoft.Xna.Framework.Graphics;
 
-namespace MonoGameExample;
+namespace ExampleMonoGame;
 
 public sealed class TextureInfo
 {