Skip to content

C# parser reads "$" as "Interpolated string expression" which is not correct #9845

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Khaos66 opened this issue Feb 9, 2023 · 4 comments
Closed
Labels

Comments

@Khaos66
Copy link

Khaos66 commented Feb 9, 2023

Describe the bug
With this C# code the method is not recognized correctly

namespace Sample.NS
{
    public class MyClass
    {
        Regex pattern = new(@".$");

        public void Fu() { }
    }
}

Expected behavior
In readme.md I need to use @ref Sample.NS.MyClass.Fu as a reference

To Reproduce
DoxygenSample.zip
In this sample the method before the ".$" string is recognized, but the second method after it is not.

Version
doxygen 1.9.6 has this issue

@Khaos66
Copy link
Author

Khaos66 commented Feb 9, 2023

@albert-github I hope the issue is setup correctly

albert-github added a commit to albert-github/doxygen that referenced this issue Feb 10, 2023
…sion" which is not correct

- the verbatim string should also be handled inside round brackets
- in case of a "normal string" inside round brackets (i.e. without the `@`) the `$"` hasn't a special meaning either, states `CopyGString` and `SkipString` should not check on `$"`.
@albert-github
Copy link
Collaborator

I've just pushed a proposed patch, pull request #9847

doxygen added a commit that referenced this issue Feb 12, 2023
issue #9845 C# parser reads "$" as "Interpolated string expression" which is not correct
@albert-github albert-github added the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Feb 12, 2023
@albert-github
Copy link
Collaborator

Code has been integrated in master on GitHub (please don't close the issue as this will be done at the moment of an official release).

@doxygen
Copy link
Owner

doxygen commented May 18, 2023

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.7.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label May 18, 2023
@doxygen doxygen closed this as completed May 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants