Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#21 Need XMLFilterImpl enhancement

closed-wont-fix
nobody
None
5
2003-07-13
2002-05-05
Bob Kerns
No

XMLFilterImpl doesn't implement the newer interfaces
such as DeclHandler or LexicalHandler.

By analogy with DefaultHandler2, an XMLFilterImpl2
could be added, but as well as I can see at this hour
of the night, this would be just so these interfaces
can remain "optional". I don't think they should be
optional going forward, so I don't see a need to
proliferate new classes to perform the same
fundamental job.

So my recommendation would be to add these interfaces
directly to XMLFilterImpl (and to DefaultHandler, and
deprecate DefaultHandler2).

Discussion

  • David Brownell
    David Brownell
    2002-05-25

    Logged In: YES
    user_id=44117

    Seems fair to me for filters to support those new interfaces.

    But at this point I'm not going to change the model where the
    "helpers" are independent of the "extensions". That did
    get discussed a while back, and the consensus was to
    not take that approach.

    If that model were to change, then "DefaultHandler2"
    would likely just vanish (as with some of the other
    classes in the "extensions 1.1 alpha").

     
  • David Brownell
    David Brownell
    2002-05-25

    • labels: 340835 -->
    • milestone: 172551 -->
    • status: open --> open-accepted
     
  • Logged In: NO

    I second Bob's frustration ... I just wanted to add a CDATA, but it is not
    trivial, since I can't just "extend" XMLFilterImpl.

     
  • Logged In: NO

    sha_aamtrading@hotmail.com

     
  • David Brownell
    David Brownell
    2003-07-13

    Logged In: YES
    user_id=44117

    Marking as "won't fix" for the reason explained in my last
    comment. It's not that it's a bad idea, though.

     
  • David Brownell
    David Brownell
    2003-07-13

    • status: open-accepted --> closed-wont-fix