DO NOT REPLY [Bug 45637] New: Add configuration option for == vs .equals() comparisons

View: New views
4 Messages — Rating Filter:   Alert me  

DO NOT REPLY [Bug 45637] New: Add configuration option for == vs .equals() comparisons

by Bugzilla from bugzilla@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

https://issues.apache.org/bugzilla/show_bug.cgi?id=45637

           Summary: Add configuration option for == vs .equals() comparisons
           Product: Security
           Version: unspecified
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Signature
        AssignedTo: security-dev@...
        ReportedBy: ashundi@...


ElementProxy initializes unconditionally the following
>
>    static ElementChecker checker = new ElementCheckerImpl.InternedNsChecker();
>
instead it should check a configuration option and initialize the appropriate
subclass of ElementCheckerImpl.

The default option should be .equals() though (change!).  Users preferring ==
should be able to configure it that way.

Correctness should trump optimization and even backward compatibility.


--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 45637] Add configuration option for == vs .equals() String comparisons

by Bugzilla from bugzilla@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

https://issues.apache.org/bugzilla/show_bug.cgi?id=45637


Anli Shundi <ashundi@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Add configuration option for|Add configuration option for
                   |== vs .equals() comparisons |== vs .equals() String
                   |                            |comparisons




--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 45637] Add configuration option for == vs .equals() String comparisons

by Bugzilla from bugzilla@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

https://issues.apache.org/bugzilla/show_bug.cgi?id=45637





--- Comment #1 from coheigea <coheigea@...>  2009-07-06 04:23:55 PST ---

I have a fix for this on a local checkout. The problem is that there are many
more pointer comparisons in the code-base, e.g. in the Canonicalizer
implementations, and I'm not comfortable adding in a configuration option for
these until I can test with a DOM implementation that does not intern
namespaces.

Is there a DOM implementation available in open-source land that either does
not intern namespaces or allows disabling of interning via configuration?

Colm.

--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 45637] Add configuration option for == vs .equals() String comparisons

by Bugzilla from bugzilla@apache.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

https://issues.apache.org/bugzilla/show_bug.cgi?id=45637

sean.mullan@... changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |elisa.laneri@...
                   |                            |.it

--- Comment #2 from sean.mullan@... 2009-09-17 12:38:28 PDT ---
*** Bug 47833 has been marked as a duplicate of this bug. ***

--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.