Skip to content
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

Whitespace until next non-whitespace token ignored after paramref in xmldoc #954

Closed
phrohdoh opened this issue Nov 16, 2016 · 2 comments
Closed

Comments

@phrohdoh
Copy link

phrohdoh commented Nov 16, 2016

Environment data

dotnet --info output:

.NET Command Line Tools (1.0.0-preview4-004081)

Product Information:
 Version:            1.0.0-preview4-004081
 Commit SHA-1 hash:  6d57ca7e14

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.12
 OS Platform: Darwin
 RID:         osx.10.11-x64

VS Code version: 1.7.1
C# Extension version: 1.5.2

Steps to reproduce

        /// <exception cref="ClientException">Thrown if <paramref name="absolutePath"/> is null, empty, or whitespace-only.</exception>
        public string PushSync(string absolutePath, object content)
        {
            ...
        }

Hover over PushSync to reveal tooltip.

Expected output

ClientException Thrown if absolutePath is null, empty, or whitespace-only.

Actual output

ClientException Thrown if absolutePathis null, empty, or whitespace-only.


Notice the missing space between absolutePath and is.

Other

I tried with both <paramref name="absolutePath"></paramref> and <paramref name="absolutePath"/>.

@DustinCampbell
Copy link
Member

Nice bug -- thanks!

@DustinCampbell
Copy link
Member

Actually, this is a duplicate of #672. The good news is that we just took a fix to OmniSharp that will fix it: OmniSharp/omnisharp-roslyn#670

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants
@DustinCampbell @phrohdoh and others