Revert "Made to mostly compile, added docs (Getting MSB4057 RN)"

This reverts commit 118fc0c.

Author: Redhacker1

src/libraries/Silk.Net.Windowing/Abstract/ContextFlags.cs

@@ -1,8 +1,4 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-using System;
-
-namespace Silk.NET.Windowing
+namespace Silk.NET.Windowing
 {
     /// <summary>
     /// Represents flags related to the OpenGL context.
@@ -27,4 +23,4 @@ public enum ContextFlags
         /// <remarks>On OpenGL contexts older than 3.0, this flag does nothing.</remarks>
         ForwardCompatible = 2
     }
-}
+}

src/libraries/Silk.Net.Windowing/Abstract/ContextProfile.cs

@@ -1,6 +1,4 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-namespace Silk.NET.Windowing
+namespace Silk.NET.Windowing
 {
     /// <summary>
     /// Represents the context profile OpenGL should use.
@@ -18,4 +16,4 @@ public enum ContextProfile
         /// </summary>
         Compatability
     }
-}
+}

src/libraries/Silk.Net.Windowing/Abstract/Delegates.cs

@@ -1,27 +1,8 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-using Silk.NET.Maths;
+using Silk.NET.Maths;
 using Silk.NET.Windowing;
 
-//TODO: Copy proposal explanation as to why we are doing this, also other than Vector2DAction these names are HIGHLY specialized for no apparently good reason, I am not a fan.
-/// <summary>
-/// An action that takes a <see cref="Vector2D{T}"/>
-/// </summary>
 public delegate void Vector2DAction(Vector2D<int> newValue);
-
-/// <summary>
-/// An action that takes a double as a parameter
-/// </summary>
 public delegate void DeltaAction(double deltaTime);
-/// <summary>
-///  An action that takes a <see cref="WindowState"/>
-/// </summary>
 public delegate void WindowStateAction(WindowState newState);
-/// <summary>
-/// An action that takes a list of strings as a parameter, intended for file path callbacks
-/// </summary>
 public delegate void FilePathsAction(string[] filePaths);
-/// <summary>
-/// An action that takes a bool as a parameter, intended for a callback when things are toggled.
-/// </summary>
-public delegate void ToggleAction(bool newValue);
+public delegate void ToggleAction(bool newValue);

src/libraries/Silk.Net.Windowing/Abstract/IDesktopSurface.cs

@@ -1,8 +1,4 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-using System;
-using System.Collections.Generic;
-using Silk.NET.Core;
+using Silk.NET.Core;
 using Silk.NET.Maths;
 
 namespace Silk.NET.Windowing
@@ -58,7 +54,7 @@ public interface IDesktopSurface : ISurface
         IEnumerable<IScreen>? AvailableScreens { get; }
 
         /// <summary>
-        /// Gets or sets whether the window waits for an event to be posted before existing <see cref="IDesktopSurface.DoEvents" />.
+        /// Gets or sets whether the window waits for an event to be posted before existing <see cref="DoEvents" />.
         /// </summary>
         bool IsEventDriven { get; set; }
 
@@ -79,7 +75,7 @@ public interface IDesktopSurface : ISurface
         /// <remarks>
         /// Because these are distances and not coordinates, they are always zero or positive.
         /// </remarks>
-        /// <seealso cref="IDesktopSurface.Size"/>
+        /// <seealso cref="WindowExtensions.GetFullSize"/>
         Rectangle<int> BorderSize { get; }
 
         /// <summary>
@@ -111,28 +107,44 @@ public interface IDesktopSurface : ISurface
         /// Sets the window icons.
         /// </summary>
         /// <param name="icons">Either a collection of window icons, or null to set to the default icon.</param>
-        void SetWindowIcon(ReadOnlySpan<RawImage> icons);
+        void SetWindowIcon(ReadOnlySpan<RawImage> icons)
+        {
+            throw new NotImplementedException();
+        }
 
         /// <summary>
-        /// When using <see cref="IDesktopSurface.IsEventDriven"/> = true, wakes the main thread from
+        /// When using <see cref="WindowOptions.IsEventDriven"/> = true, wakes the main thread from
         /// its blocking wait on incoming events.  Can be called from any thread.
         /// </summary>
-        void ContinueEvents();
+        void ContinueEvents()
+        {
+            throw new NotImplementedException();
+        }
 
         /// <summary>
         /// Converts this point to client coordinates.
         /// </summary>
         /// <param name="point">The point to transform.</param>
         /// <returns>The transformed point.</returns>
         /// <remarks>Expects screen coordinates as input.</remarks>
-        Vector2D<int> PointToClient(Vector2D<int> point);
+        Vector2D<int> PointToClient(Vector2D<int> point)
+        {
+            return new Vector2D<int>(point.X - Position.X, point.Y - Position.Y);
+        }
 
         /// <summary>
         /// Converts this point to screen coordinates.
         /// </summary>
         /// <param name="point">The point to transform.</param>
         /// <returns>The transformed point.</returns>
         /// <remarks>Expects client coordinates as input.</remarks>
-        Vector2D<int> PointToScreen(Vector2D<int> point);
+        Vector2D<int> PointToScreen(Vector2D<int> point)
+        {
+            return new Vector2D<int>(point.X + Position.X, point.Y + Position.Y);
+        }
+        public Vector2D<int> PointToFramebuffer(Vector2D<int> point)
+        {
+            throw new NotImplementedException();
+        }
     }
-}
+}

src/libraries/Silk.Net.Windowing/Abstract/IGLDesktopSurface.cs

@@ -1,9 +1,5 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
+
 namespace Silk.NET.Windowing
 {
-    /// <summary>
-    /// An interface that defines a desktop surface with the ability to create a desktop GL Context
-    /// </summary>
     public interface IGLDesktopSurface : IDesktopSurface, IGLSurface { }
-}
+}

src/libraries/Silk.Net.Windowing/Abstract/IGLSurface.cs

@@ -1,28 +1,17 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-namespace Silk.NET.Windowing
+namespace Silk.NET.Windowing
 {
-    /// <summary>
-    /// A surface that can have a desktop GL context.
-    /// </summary>
     public interface IGLSurface : INativeGLSurface
     {
-        /// <summary>
-        /// The flags the context was created with
-        /// </summary>
         ContextFlags ContextFlags { get; set; }
-        /// <summary>
-        /// The profile of the specified context.
-        /// </summary>
         ContextProfile ContextProfile { get; set; }
-        /// <summary>
-        /// The surface representing the context to use.
-        /// </summary>
-        IGLSurface? SharedContext { get; set; } // Redhacker1/Donovan: DAFUQ? What exactly is supposed to be going on here
+        IGLSurface? SharedContext { get; set; }
 
         /// <summary>
         /// Enables OpenGL support for this surface. This will create a surface upon initialization.
         /// </summary>
-        bool TryEnableOpenGL();
+        bool TryEnableOpenGL()
+        {
+            throw new NotImplementedException();
+        }
     }
-}
+}

src/libraries/Silk.Net.Windowing/Abstract/IGLTransparentFramebufferSurface.cs

@@ -1,16 +1,8 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
+
 namespace Silk.NET.Windowing
 {
-    /// <summary>
-    /// Represents a GL Surface with a transparent Framebuffer
-    /// </summary>
-    // Is GLFW making this an optional feature worth adding this? -Donovan/Redhacker1
     public interface IGLTransparentFramebuffer : INativeGLSurface
     {
-        /// <summary>
-        /// Whether the buffer can be actually transparent
-        /// </summary>
         bool TransparentFramebuffer { get; set; }
     }
-}
+}

src/libraries/Silk.Net.Windowing/Abstract/IGlesDesktopSurface.cs

@@ -1,12 +1,7 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-namespace Silk.NET.Windowing
+namespace Silk.NET.Windowing
 {
-    /// <summary>
-    /// A Desktop Surface with the ability to create a GLES surface
-    /// </summary>
     public interface IGlesDesktopSurface : IDesktopSurface, IGlesSurface
     {
 
     }
-}
+}

src/libraries/Silk.Net.Windowing/Abstract/IGlesSurface.cs

@@ -1,19 +1,11 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-namespace Silk.NET.Windowing
+namespace Silk.NET.Windowing
 {
-    /// <summary>
-    /// Defines a surface that can have a GLES context attatched
-    /// </summary>
     public interface IGlesSurface : INativeGLSurface
     {
-        /// <summary>
-        /// The surface representing the context to use.
-        /// </summary>
-        IGlesSurface? SharedContext { get; set; } // Donovan/Redhacker1: How on earth is this supposed to work?
+        IGlesSurface? SharedContext { get; set; }
         /// <summary>
         /// Enables OpenGLES support for this surface. This will create a surface upon initialization.
         /// </summary>
         bool TryEnableOpenGLES();
     }
-}
+}

src/libraries/Silk.Net.Windowing/Abstract/INativeGLSurface.cs

@@ -1,23 +1,12 @@
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-using Silk.NET.Core;
+using Silk.NET.Core;
 using Silk.NET.Maths;
 
 namespace Silk.NET.Windowing
 {
     public interface INativeGLSurface : ISurface
     {
-        /// <summary>
-        /// the GL Handle
-        /// </summary>
         nint Handle { get; }
-        /// <summary>
-        /// Whether this is the currently running context on this thread
-        /// </summary>
         bool IsContextCurrent { get; set; }
-        /// <summary>
-        /// Should the buffers swap immediately upon completion?
-        /// </summary>
         bool ShouldSwapAutomatically { get; set; }
 
         /// <summary>
@@ -59,6 +48,9 @@ public interface INativeGLSurface : ISurface
         Version32? ApiVersion { get; set; }
 
         nint? GetProcAddress(string proc);
-        void SwapBuffers();
+        void SwapBuffers()
+        {
+            throw new NotImplementedException();
+        }
     }
-}
+}