https://defect.opensolaris.org/bz/show_bug.cgi?id=16936

Knut Anders Hatlen <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #1 from Knut Anders Hatlen <[email protected]> 2010-08-27 
08:23:17 UTC ---
I noticed the same issue when I fixed bug #3435, and I added the following
comment in JFlexXref.java:

http://src.opensolaris.org/source/xref/opengrok/trunk/src/org/opensolaris/opengrok/analysis/JFlexXref.java#154

          // 1) Create an anchor for direct links. (Perhaps, we should only
          //    do this when there's exactly one definition of the symbol in
          //    this file? Otherwise, we may end up with multiple anchors with
          //    the same name.)

We no longer generate links that point to these ambiguous anchors (at least we
shouldn't after bug #3435), but we still generate the anchors in case someone
has old links lying around that would break if we remove them. I suppose that's
not much of an issue, so we could probably stop generating those anchors
altogether. Or perhaps add a counter to the anchor name if there are multiple
definitions with the same name.

Of course, it would be better if we could distinguish between the different
signatures and provide direct links from the references to the correct
definition when there are many with the same name. However, we don't have the
infrastructure in place to do that (missing number of arguments and type
information), so I think that would require a larger effort.

-- 
Configure bugmail: https://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
opengrok-dev mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/opengrok-dev

Reply via email to