From d306115750df9df170cfef4d49c6b0b7af498962 Mon Sep 17 00:00:00 2001
From: jduncanator <1518948+jduncanator@users.noreply.github.com>
Date: Mon, 11 Feb 2019 23:00:32 +1100
Subject: Logger and Configuration Refactoring (#573)

* Logging: Refactor log targets into Ryujinx.Common

* Logger: Implement JSON Log Target

* Logger: Optimize Console/File logging targets

Implement a simple ObjectPool to pool up StringBuilders to avoid causing excessive GCing of gen1/2 items when large amounts of log entries are being generated.

We can also pre-determine the async overflow action at initialization time, allowing for an easy optimization in the message enqueue function, avoiding a number of comparisons.

* Logger: Implement LogFormatters

* Config: Refactor configuration file and loading

* Config: Rename to .jsonc to avoid highlighting issues in VSC and GitHub

* Resolve style nits

* Config: Resolve incorrect default key binding

* Config: Also update key binding default in schema

* Tidy up namespace imports

* Config: Update CONFIG.md to reflect new Config file
---
 Ryujinx.Common/Pools/ObjectPool.cs | 75 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 75 insertions(+)
 create mode 100644 Ryujinx.Common/Pools/ObjectPool.cs

(limited to 'Ryujinx.Common/Pools/ObjectPool.cs')

diff --git a/Ryujinx.Common/Pools/ObjectPool.cs b/Ryujinx.Common/Pools/ObjectPool.cs
new file mode 100644
index 00000000..dba671bb
--- /dev/null
+++ b/Ryujinx.Common/Pools/ObjectPool.cs
@@ -0,0 +1,75 @@
+using System;
+using System.Threading;
+
+namespace Ryujinx.Common
+{
+    public class ObjectPool<T>
+        where T : class
+    {
+        private T _firstItem;
+        private readonly T[] _items;
+
+        private readonly Func<T> _factory;
+
+        public ObjectPool(Func<T> factory, int size)
+        {
+            _items   = new T[size - 1];
+            _factory = factory;
+        }
+
+        public T Allocate()
+        {
+            var instance = _firstItem;
+
+            if (instance == null || instance != Interlocked.CompareExchange(ref _firstItem, null, instance))
+            {
+                instance = AllocateInternal();
+            }
+
+            return instance;
+        }
+
+        private T AllocateInternal()
+        {
+            var items = _items;
+
+            for (int i = 0; i < items.Length; i++)
+            {
+                var instance = items[i];
+
+                if (instance != null && instance == Interlocked.CompareExchange(ref items[i], null, instance))
+                {
+                    return instance;
+                }
+            }
+
+            return _factory();
+        }
+
+        public void Release(T obj)
+        {
+            if (_firstItem == null)
+            {
+                _firstItem = obj;
+            }
+            else
+            {
+                ReleaseInternal(obj);
+            }
+        }
+
+        private void ReleaseInternal(T obj)
+        {
+            var items = _items;
+
+            for (int i = 0; i < items.Length; i++)
+            {
+                if (items[i] == null)
+                {
+                    items[i] = obj;
+                    break;
+                }
+            }
+        }
+    }
+}
-- 
cgit v1.2.3