Function list: fix Java parsing and add C# support

[hybridmachine]

Summary

The function browser missed most Java methods, and C# wasn't a supported language at all. This fixes Java parsing and adds full C# support (highlighting + function list).

Java was broken

The Java regex required ) immediately followed by {, so on a representative file it found only 2 of 7 methods. It missed:

• throws clauses — void readFile() throws IOException {
• Same-line annotations — @Override public String toString() {
• Allman braces — signature on one line, { on the next (the Java path had no pending-signature handling; C++ did)

It also listed control statements (if/while/try) as methods.

C# was absent

No LANG_CSHARP, no .cs detection, no language-table entry — so C# files got no syntax highlighting and an empty function list.

Changes

• function_list_parser.mm — rework the Java path into a shared Java/C# branch: annotations, throws, Allman pending-signatures, C# expression-bodied members (=>), : base(...) constructors, where constraints, and class|struct|interface|enum|record containers. Control keywords filtered via a separate set so lock/using stay valid C++ names. Regexes match modifiers/types token-wise behind cheap string prefilters.
• language_defs.{h,mm} — add LANG_CSHARP = 36 (appended at the end of the table, since sessions persist languageIndex as a raw int), C# keyword/type lists on the cpp lexer (as upstream Notepad++ does), .cs/.csx detection, C-style indentation.
• npp_constants.h — move IDM_LANG_BASE 44000 → 51000. This fixes a latent bug: 44000+ is upstream's plugin-visible view-command range, and the language range check runs first in MainWndProc, so 44035 (IDM_VIEW_SYNSCROLLV, sent by ComparePlus) was misrouted to "set language to x86 Assembly". The move also makes room for the 37th language.

Verification

• Parser harness (real parser compiled standalone): comprehensive Java/C# samples pass; C++/Python output is byte-identical to the old parser on regression inputs.
• Performance: the first regex draft backtracked badly (195s on a 127k-line C# file). Restructured to token-wise matching + prefilters → ~5s on 115k+ line Java/C# files (the old parser took 57s on the same Java input). Typical files parse in milliseconds.
• Live app: C# appears in the Language menu; Sample.cs renders the full C# tree (classes, ctors incl. : base(...), async methods, expression-bodied members — properties correctly excluded); Java.java renders all expected rows including every previously-missing pattern.

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR improves the macOS function list browser by fixing Java method detection and adding first-class C# support (syntax highlighting via the existing cpp lexer plus function/container extraction). It also moves the language-menu command-ID range to avoid collisions with upstream Notepad++ command IDs that plugins may send by value.

Changes:

• Move IDM_LANG_BASE to a safe range to prevent plugin-sent upstream view commands (e.g., sync scroll) from being misrouted as “set language”.
• Add LANG_CSHARP, C# language table entry, .cs/.csx detection, and mark C# as C-style indentation.
• Rework the function list parser to share a Java/C# parsing path with annotation/throws handling, Allman-style pending signatures, and C#-specific syntax forms.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
macos/platform/npp_constants.h	Moves language command-ID base away from upstream plugin command ranges; adds explanatory comments.
macos/platform/language_defs.mm	Adds C# language definition, extension detection, and C-style indentation classification.
macos/platform/language_defs.h	Introduces LANG_CSHARP language index.
macos/platform/function_list_parser.mm	Adds Java/C# shared parsing branch and new regexes/filters for more accurate container + function extraction.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[hybridmachine]

Confirmed — the vendored menuCmdID.h tops out at IDM_EDIT_FUNCCALLTIP_NEXT = 50011 (autocomplete block at 50000+). Comment corrected in 744d028; the 51000 base itself is unaffected since nothing upstream sits in 51000–51100 (verified by resolving every #define in the header).

[hybridmachine]

Good catch — confirmed with a failing test: record struct Coordinate(...) produced a container literally named "struct". Fixed in 744d028 by matching record class/record struct as a unit in javaCsClassRe (longest alternative first, name capture unchanged). Harness now covers record struct and readonly record class with methods grouped under the right containers.