On Tue, 5 Jun 2001 23:54:36 -0400, [EMAIL PROTECTED] wrote:
> I think the Node compatibility interface. Opinions?
I would propose to have both the interfaces exposed.
We can have the DTM interface exposed in addition to the Node
interface in the SelectionEvent, GenerateEvent, and TracerEvent
classes through public methods. We would need to hide the Node
m_sourceNode instance variable, which may be a bad thing from a
compatibility perspective.
So, leaving aside the compatibility issue, the above classes will have
the m_sourceNode public instance variable removed, and instead the
node handle added (maybe as a public ivar). The following methods will
be then added:
public int getSourceNodeHandle();
public Node getSourceNode();
The last method is actually doing the conversion from the node handle
to the actual Node instance.
This approach however exposes the DTM API, which may not be a good
thing, but for the sake of speed and memory this should happen
anyway. Another issue is that we are no longer compatible with the old
API (argh, bad decision to expose the internals of the trace classes
as public members).
> -scott
Regards,
--
Ovidiu Predescu <[EMAIL PROTECTED]>
http://orion.nsr.hp.com/ (inside HP's firewall only)
http://www.geocities.com/SiliconValley/Monitor/7464/ (GNU, Emacs, other stuff)
>
>
>
>
>
>
> Ovidiu
>
>
> Predescu To: [EMAIL PROTECTED]
>
>
> <[EMAIL PROTECTED] cc: (bcc: Scott Boag/CAM/Lotus)
>
>
> p.com> Subject: [DTM_EXP] TraceListener
>interface
>
> Sent by:
>
>
> ovidiu@orion.
>
>
> nsr.hp.com
>
>
>
>
>
>
>
>
> 06/04/2001
>
>
> 06:38 PM
>
>
> Please
>
>
> respond to
>
>
> xalan-dev
>
>
>
>
>
>
>
>
>
>
>
>
> I've just checked the TraceListener interface and it appears to me
> that the API uses the Node interface. Is this going to change to use
> the new DTM interface or will we be still using the Node compatibility
> API from the DTM interface?
>
> Greetings,
> --
> Ovidiu Predescu <[EMAIL PROTECTED]>
> http://orion.nsr.hp.com/ (inside HP's firewall only)
> http://www.geocities.com/SiliconValley/Monitor/7464/ (GNU, Emacs, other
> stuff)
