|
From: Andreas L. <no...@sb...> - 2001-10-22 21:58:57
|
On Mon, 2001-10-22 at 23:43, Andreas Leitner wrote: > On Mon, 2001-10-22 at 15:31, Berend de Boer wrote: > > Andreas Leitner <no...@sb...> writes: > > > > > > I assume XI_PARSER is some base class here?? I don't see another need > > > > for inheritance in this case, it's just a class with a callback > > > > feature. And it doesn't need anything currently in XI_PARSER, it's > > > > just confusing IMO. > > > > > > XI_PARSER defines the features like `parse_from_string', > > > `parse_from_buffer', `parse_incremental_from_string', ... > > > > But you don't need them in the callback class. Perhaps move > > parse_from_XXXX to XI_REAL_PARSER_BASE or so. > > That's why XI_CALLBAKCS would not inherit from XI_PARSER. The parser is > the source, the callbacks are the sink. > > Frank, are you with us on this approach? Or do you favor yours? > > Andreas > > |