FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Restore "Fuck unicode" comment in JsonParser.cs by Copilot · Pull Request #53 · jokedst/CsvQuery · GitHub

Repository navigation

Restore "Fuck unicode" comment in JsonParser.cs - #53

Draft
jokedst with Copilot wants to merge 4 commits into
masterfrom
copilot/suggest-codebase-improvements
Draft

jokedst with Copilot wants to merge 4 commits into
masterfrom
copilot/suggest-codebase-improvements

Conversation

Copilot AI commented Mar 4, 2026 •
edited
Loading

Copy link
Copy Markdown

The comment // Fuck unicode was removed in a prior cleanup pass. The repo owner considers it factually correct and wants it kept.

  • JsonParser.cs: Restored // Fuck unicode above the \uXXXX escape handler, which truncates full Unicode code points to a single char — a real limitation for characters outside the BMP.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits March 4, 2026 19:53
…e error logging

Co-authored-by: jokedst <858348+jokedst@users.noreply.github.com>
Co-authored-by: jokedst <858348+jokedst@users.noreply.github.com>
Copilot AI changed the title [WIP] Analyze codebase and suggest improvements Code quality improvements: variance bug fix, resource leaks, dead code, and constants Mar 4, 2026
Co-authored-by: jokedst <858348+jokedst@users.noreply.github.com>
Copilot AI changed the title Code quality improvements: variance bug fix, resource leaks, dead code, and constants Restore "Fuck unicode" comment in JsonParser.cs Mar 4, 2026

This branch has not been deployed

No deployments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL