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

[READY] Handle multiple identical diagnostics in YcmShowDetailedDiagnostic popup #4219

Merged

Commits on Feb 3, 2024

  1. Handle multiple identical diagnostics in YcmShowDetailedDiagnostic popup

    If a line for which detailed diagnostics are requested contains
    more than a single diagnostic with the same message, YCM will try
    `options.pop( 'col' )` more than once.
    
    We do not need to really iterate through the diagnostics once we have
    found the first one that is a match. If there's only one diag with the
    matching message, looping beyond that diagnostic is just a waste of
    time. If there's multiple diagnostics with the same message, it does not
    matter which one we display in the popup.
    
    Hence, the added break at the end of the loop.
    bstaletic committed Feb 3, 2024
    Configuration menu
    Copy the full SHA
    cea2458 View commit details
    Browse the repository at this point in the history