New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 600975 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Incorrect C++ parsing in Chromium's code search

Project Member Reported by carlosk@chromium.org, Apr 6 2016

Issue description

I was looking at navigation_params.h [1] in CodeSearch and I noticed its outline view was very incomplete. Looking closer I noticed that many of the structs defined in that file are being presented as methods. And for one of them there's an extra error where the username from a TODO comment from the line following the struct declaration was somehow considered a parameter for the "method".

See attached image.

[1] https://code.google.com/p/chromium/codesearch#chromium/src/content/common/navigation_params.h
 
codesearch_wrong_outline.png
67.8 KB View Download
Cc: dsansome@chromium.org

Comment 2 by benhenry@google.com, Apr 27 2016

Components: Infra>Codesearch
Labels: -Infra-Codesearch
Status: Available (was: Untriaged)

Comment 4 by emso@chromium.org, Sep 15 2016

Labels: Lang Visualization
Project Member

Comment 5 by sheriffbot@chromium.org, Sep 15 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 6 by emso@chromium.org, Sep 18 2017

Status: Available (was: Untriaged)
Project Member

Comment 7 by sheriffbot@chromium.org, Sep 18

Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Recharge-Cold
Status: Available (was: Untriaged)

Sign in to add a comment