⚡ Optimize string concatenation and logic in Citra 3DS Manager

[Ven0m0]

💡 What:

• Introduced a remCount early-exit guard to skip dictionary lookups (InStr, HasKey) when all configuration updates are satisfied.
• Replaced multi-part chained string concatenation (var .= a b c) with single-part consecutive assignments (var .= a, var .= b).

🎯 Why:

• While the script already correctly utilized VarSetCapacity prior to the loop to allocate memory, it was using chained .= within the parsing loop, which inherently causes AutoHotkey v1 to allocate tiny temporary strings prior to the main append operation.
• The original logic forced an InStr and HasKey lookup for every single line of the configuration file, even if all modifications had already been applied near the top of the file.

📊 Measured Improvement:
Note: As this system operates in a headless Linux environment, direct benchmark execution against the native AHK v1 interpreter was not feasible. However, theoretical runtime complexity checks map directly to established AHK v1 optimization best practices.

• Lookup logic transitions from O(N) to O(K) where N is file lines and K is the subset of lines until all updates are satisfied. This yields faster processing, especially for larger config files with updates concentrated near the start.
• Single-part string assignment explicitly bypasses the well-documented v1 temporary variable intermediate allocations, reducing memory churn.

---

PR created automatically by Jules for task 12318181205727613221 started by @Ven0m0

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[gemini-code-assist]

Code Review

This pull request optimizes the UpdateConfig function in the legacy AutoHotkey v1 script Citra_3DS_Manager.ahk by reducing temporary string allocations during file parsing and finalization. It also introduces a counter to skip key-matching logic once all updates have been processed. Feedback suggests further optimizing a specific line within the parsing loop to maintain consistency with the rest of the performance-oriented changes.

[gemini-code-assist]

To maintain consistency with the optimization goal of avoiding temporary string allocations in AutoHotkey v1, this chained assignment should be split into separate operations. While this line executes at most once per update, it currently creates a temporary string for the expression keyCandidate "=" val before assignment, which contradicts the rationale provided in the PR description for other similar changes in the parsing and finalization loops.

          line := keyCandidate
          line .= "="
          line .= val

[Copilot]

Pull request overview

This PR updates the legacy AutoHotkey v1 Citra 3DS Manager config-rewrite routine to reduce per-line work during config parsing and to avoid multi-part chained .= concatenations.

Changes:

• Adds a remCount guard to skip InStr/HasKey work once all requested key updates have been applied.
• Replaces single-expression concatenations like newContent .= line "n" with consecutive appends (newContent .= linethennewContent .= "n").