Re: DSML library version .01

[Mark W. <mew...@un...>]

Graham Barr wrote:

> On Thu, May 25, 2000 at 08:01:11PM -0500, Mark Wilcox wrote:
> > Hi,
> > Here's my first version of Net::DSML (and the helper module XML::DSML
> > which you could use seperately, but it's probably easier to use
> > Net::DSML).
>
> This is a great start.

Thanks. Last night I noticed a bug in my Base64 code for Net::LDAP::DSML, so I'm shipping
a new copy.

Also I've made a patch to Net::LDAP::Entry so that the dump() method will Base64 encocde
binary in the ouptut.

>
>
> In XML::DSML you have a comment about renaming the module as Net::LDAP::DSML::Parser.
>
> My answer would be that if it is a generic DSML parser, not connected with Net::LDAP,
> then XML::DSML is probably the right place for it to live.

Yeah that's what I'm thinking as well, but I wanted to get something out for people to
play with and then figure out the best way to go.

>
>
> I have not tried the code, but from looking at it I have one concern.
> It seems that the whole DSML file must be read into memory in one go.

I'll have to double-check this. I can't remember if XML::Parser's callback functions use
streaming or load the whole file.

>
>
> This will be a disadvantage for large files and we should probably devise a way
> to read the entries one at a time.

I'll double check, but worst-case I can rewrite it so that XML::Parser streams in the
file.

>

Mark

>

>
>
> I guess you did it this way as by default XML::Parser works on the whole
> file before it returns. But IIRC, it is possible to interface to
> XML::Parser at a lower level. If not them maybe XML::Parser is not the
> way to go.

>
>
> Graham.