From: Graham B. <gb...@po...> - 2000-05-26 08:07:26
|
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. 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. 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. This will be a disadvantage for large files and we should probably devise a way to read the entries one at a time. 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. |