184338 - lxr Ident search not finding all occurences of identifier

Status: RESOLVED WONTFIX

(Webtools Graveyard :: MXR, defect)

Description

[Nathan Kurz]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2b) Gecko/20021029 Phoenix/0.4
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2b) Gecko/20021029 Phoenix/0.4

Search by identifier for "NS_CheckContentLoadPolicy":
http://lxr.mozilla.org/seamonkey/ident?i=NS_CheckContentLoadPolicy
   Referenced in six files.

Search by string for "NS_CheckContentLoadPolicy":
http://lxr.mozilla.org/seamonkey/search?string=NS_CheckContentLoadPolicy
   Finds occurences in seven files. 

The first two occurences in docshell/base/nsDocShell.cpp are not found when
searching by identifier.  But "NS_CheckContentLoadPolicy" is highlighted and
acts like an identifier from within that file.  




Reproducible: Always

Steps to Reproduce:

Comment 1

[timeless]

strange. it wouldn't be so bad if it ignored the comment entry and hit the
other, or if it ignored the entire file, but it knows about the file
http://lxr.mozilla.org/seamonkey/ident?i=SameOrSubdomainOfTarget and it knows
about the identifier... note that the file hasn't changed in about a week either...

Comment 2

[Ben Fowler]

This is a 'true bill'. A search for the identifier
<URL: http://lxr.mozilla.org/seamonkey/ident?i=p2cstr > comes
back with nothing, but the comparable text search
<URL: http://lxr.mozilla.org/seamonkey/search?string=p2cstr >
picked up what I was looking for. This may have been a C pre-processing
issue .

It could be that a more focussed parser is required, and this is
hardly within Mozilla's purview.
<URL: http://lxr.linux.no/ >

Comment 3

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Is this dupe of bug 54995?

Comment 4

[Brodie]

Same problem here...

http://lxr.mozilla.org/seamonkey/search?string=nsIStandardURL
== lots of hits

http://lxr.mozilla.org/seamonkey/ident?i=nsIStandardURL
== no hits

Comment 5

[timeless]

heikki: this is not a dupe of that older bug, i can reproduce these...

ben: our lxr has its roots in the other lxr, but has significantly diverged from
it and is maintained (as best as I can) independently. note that in your choice,
there is *no* definition of that function, so i'd say that your problem is
probably different.

brodie: that problem is actually different, the declaration of nsIStandardURL is
in an idl file, which lxr doesn't understand. there's supposed to be bug/patch
that attempts to deal w/ interfaces, but clearly it doesn't work (because
http://landfill.mozilla.org/mxr-test/seamonkey/ident?i=nsIStandardURL doesn't
show anything :( ...)

Comment 6

[Dave Miller [:justdave]]

reassigning endico's bugs to default owner

Comment 7

[Chase Phillips]

Mass reassign of open bugs for [e-mail] to [e-mail].

Comment 8

[J. Paul Reed [:preed]]

Reassigning all LXR bugs assigned to [e-mail] to the default LXR owner (sorry Bear!)

We aren't actively working on these (if that's wrong, please reassign to yourself/a real person). 

Comment 9

[Sylvestre Ledru [:Sylvestre]]

mxr is gone, mass closing.
https://searchfox.org/ is a much better alternative.