Skip to content

Merge from main#143

Merged
fredvs merged 20 commits intoX11_cleanfrom
main
Feb 6, 2026
Merged

Merge from main#143
fredvs merged 20 commits intoX11_cleanfrom
main

Conversation

@fredvs
Copy link
Contributor

@fredvs fredvs commented Feb 6, 2026

No description provided.

fredvs and others added 20 commits August 2, 2025 05:27
Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 5.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v4...v5)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…s/checkout-5

Bump actions/checkout from 4 to 5
Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5...v6)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…s/checkout-6

Bump actions/checkout from 5 to 6
When navigating with Ctrl+Click to namespace units like FPTest.Core.Utils,
the IDE was unable to find the unit because the parser was stopping at dots
when parsing the uses clause.

Changes:

1. Removed dot from link delimiters in showlink (sourcepage.pas:1206)
   - This allows the entire namespace unit name to be underlined when hovering
   - Dots are now not treated as word boundaries for navigation highlighting

2. Fixed getorignamelist to handle namespace units (mseparser.pas:1794)
   - The parser now concatenates dotted namespace parts (Something.Utils)
   - Uses testoperator to detect dots without consuming the token
   - Properly extends the end position to encompass the entire namespace unit

The root cause was that getorignamelist would stop parsing at the first dot
operator, returning only the first part of a namespace unit. This caused
findsourceitem to fail because the recorded endpos was before the cursor
position when clicking on the latter parts of the namespace unit name.
==SUMMARY==
Branch bug/namespace-navigation commits:
* Detect when we are in the uses section or elsewhere
* Fix code navigation for namespace units (issue #138)

Branch main commits since creating feature branch:

==DETAILS==
commit + 8a49e32
Author: Graeme Geldenhuys <graemeg@gmail.com>
Date:   Sat Dec 13 10:12:19 2025 +0000

    Detect when we are in the uses section or elsewhere

commit + d042826
Author: Graeme Geldenhuys <graemeg@gmail.com>
Date:   Fri Dec 12 12:59:27 2025 +0000

    Fix code navigation for namespace units (issue #138)

    When navigating with Ctrl+Click to namespace units like FPTest.Core.Utils,
    the IDE was unable to find the unit because the parser was stopping at dots
    when parsing the uses clause.

    Changes:

    1. Removed dot from link delimiters in showlink (sourcepage.pas:1206)
       - This allows the entire namespace unit name to be underlined when hovering
       - Dots are now not treated as word boundaries for navigation highlighting

    2. Fixed getorignamelist to handle namespace units (mseparser.pas:1794)
       - The parser now concatenates dotted namespace parts (Something.Utils)
       - Uses testoperator to detect dots without consuming the token
       - Properly extends the end position to encompass the entire namespace unit

    The root cause was that getorignamelist would stop parsing at the first dot
    operator, returning only the first part of a namespace unit. This caused
    findsourceitem to fail because the recorded endpos was before the cursor
    position when clicking on the latter parts of the namespace unit name.
Update link to Bash installer
@fredvs fredvs merged commit 997c510 into X11_clean Feb 6, 2026
5 checks passed
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.

3 participants