[xslt] Re: Re: Work report/code review part 3

[k_isdale tpg com au]

Hi all,

In the last post Daniel said 

> Just some comments about my last post. 
> Yes the breakpoint.[ch] was intended to go into 
> the libxslt/ of libxslt library. 

>             And I'm not sure it's actaully the 
right place.

That fine by me. In libxslt/,  transform.c only 
needs to have the xslHandleDebugger function. All 
other breakpoint code can go into a separate 
library. I hadn't planed to "breakup" 
breakpoints.[ch] in multiple files. But if the 
breakpoint API is to be in a separate library 
maybe that would be a good thing to do. What do 
you think? A possible "breakup" would be 
dbgmain.[ch], dbgcallstack.[ch], 
dbgbreakpoint.[ch]. For backward compatability we 
could keep breakpoint.h and have it include all 
breakpoint API related headers.

For the header/implementation for 
xslHandleDebugger see libxslt-1.0.5_ki_0.6.2.diff 
(line 1529-1599 and line 1789 - 1817) in 
http://prdownloads.sourceforge.net/xsldbg/libxslt-
1.0.5_diff.tar.gz

bye,

Keith