[PR #63] [CLOSED] Performance upgrades, prevention of potential crashing #969

Closed
opened 2026-02-27 15:52:39 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/quasar/Quasar/pull/63
Author: @yankejustin
Created: 4/20/2015
Status: Closed

Base: masterHead: master


📝 Commits (7)

  • 11d9f10 Removed unnecessary code and improvements
  • 2da21b8 Code consistency and safety
  • f1a9f0f Memory leaks, safety, and consistent behavior
  • 8204e6a Reflected changes to Server's UnsafeStreamCodec
  • f328d37 Removed unnecessary variables
  • 77f1af8 Prioritized packet reading for the client
  • 750aede Merge pull request #1 from yankejustin/Experimental

📊 Changes

8 files changed (+226 additions, -172 deletions)

View changed files

📝 Client/Core/Client.cs (+1 -1)
📝 Client/Core/Helper/UnsafeStreamCodec.cs (+34 -55)
📝 Client/Program.cs (+18 -18)
📝 Server/Core/Client.cs (+1 -1)
📝 Server/Core/Helper/UnsafeStreamCodec.cs (+33 -55)
📝 Server/Core/UserState.cs (+28 -2)
📝 Server/Forms/FrmFileManager.cs (+46 -23)
📝 Server/Forms/FrmMain.cs (+65 -17)

📄 Description

Formatting changes, slight memory leak fixes, removal of unnecessary of old and commented-out code, readability improvements, consistency improvements, removal of two unnecessary variables, and some optimization.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/quasar/Quasar/pull/63 **Author:** [@yankejustin](https://github.com/yankejustin) **Created:** 4/20/2015 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (7) - [`11d9f10`](https://github.com/quasar/Quasar/commit/11d9f10408fbe5457a0a983dbd663b5890b2e3bf) Removed unnecessary code and improvements - [`2da21b8`](https://github.com/quasar/Quasar/commit/2da21b83df4d83fbeabf810b981239b8ca8942ff) Code consistency and safety - [`f1a9f0f`](https://github.com/quasar/Quasar/commit/f1a9f0f3ca8ca52b01b6949bd56137c8f4435752) Memory leaks, safety, and consistent behavior - [`8204e6a`](https://github.com/quasar/Quasar/commit/8204e6a19b86dbe409bd3a59ecde5298f5a9c0a9) Reflected changes to Server's UnsafeStreamCodec - [`f328d37`](https://github.com/quasar/Quasar/commit/f328d376e8c12782492c1882b44433afa7b9c23f) Removed unnecessary variables - [`77f1af8`](https://github.com/quasar/Quasar/commit/77f1af8e6143e5bc8b62613e5f94b2c7909d4261) Prioritized packet reading for the client - [`750aede`](https://github.com/quasar/Quasar/commit/750aede976ca933f093157faa6a29e501d2c7d34) Merge pull request #1 from yankejustin/Experimental ### 📊 Changes **8 files changed** (+226 additions, -172 deletions) <details> <summary>View changed files</summary> 📝 `Client/Core/Client.cs` (+1 -1) 📝 `Client/Core/Helper/UnsafeStreamCodec.cs` (+34 -55) 📝 `Client/Program.cs` (+18 -18) 📝 `Server/Core/Client.cs` (+1 -1) 📝 `Server/Core/Helper/UnsafeStreamCodec.cs` (+33 -55) 📝 `Server/Core/UserState.cs` (+28 -2) 📝 `Server/Forms/FrmFileManager.cs` (+46 -23) 📝 `Server/Forms/FrmMain.cs` (+65 -17) </details> ### 📄 Description Formatting changes, slight memory leak fixes, removal of unnecessary of old and commented-out code, readability improvements, consistency improvements, removal of two unnecessary variables, and some optimization. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 15:52:39 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/Quasar#969
No description provided.