Karl Dahlke <[email protected]> writes:

> but why aren't you using
>
> static list < struct htmlTag >htmlStack;

That's not a bad idea.  There are a few things preventing it.
htmlTag should probably be a proper class, with a constructor,
destructor, copy constructor, and assignment operator.
Also, to make life easier, all the char * members need to be replaced
with string.  I'm working on that, but it'll take a while.

Next, we have an issue of duplication.  We have two data structures that
will hold the same set of tags, one being cw->tags (an array), and the
other being htmlStack.  If we use structs instead of pointers in these,
the data structures can get out of sync when code modifies a tag in just
one of them.  If cw->tags is never constructed while htmlStack is in
use, then this isn't actually a problem.  But I'm not sure if that is
the case.

-- Chris
_______________________________________________
Edbrowse-dev mailing list
[email protected]
http://lists.the-brannons.com/mailman/listinfo/edbrowse-dev

Reply via email to