Re: problem using INTERFACE_BUFFER

David Relson wrote:
> 
> At 11:32 AM 12/23/02, [email protected] wrote:
> 
> >Yeah.  Version 1.4 fixes all allocation/deallocation errors, as well
> >as doing away with the internal MEM_DEBUG stuff (uses an external
> >library now).  I'll put 1.4 up pre-release.  It's not done yet, but it
> >compiles and works.
> >
> >Check the link periodically http://www.inter7.com/eps/eps-1.4.tar.gz
> >I should have it up in an hour or so.
> >
> >Good luck!
> 
> Hi Matt,
> 
> FWIW, looking over the code I found a number of interesting things to do to
> it.
> 
> Debugging can be made easier by using enumerated types (which display as
> names) instead of #defines (which display as numbers).  For example, in
> content.h I have "typedef enum enc_e { ... } enc_t" and in mime.h I have
> distinct types in the struct mime_t definition.

By all means, please provide any patches you can come up with.  In
general,
however, the current development version takes into account suggestions
from many people.  EPS's development is getting slower now as it has
become
more feature-rich for what it was started for (eFilter).  I have less
time
to work on EPS since it is no longer scheduled time during my day here,
but I gather Ill be working on EPS for the next year or so off and on.

> 
> In several places there are "if () elseif () elseif()..." sequences that
> can be nicely replaced by switches.  eps.c, content.c, and encode.c are
> places where I've done this.

Well, the question here becomes (for me at least; since this is a parser
library),
which is going to be the most efficient, or is there no difference after
the compiler interprets the code?

> 
> Stepping through arrays of structures can be made more readable by using
> pointers rather than an index and subscripting.  Similarly a for() loop
> (with its initialization, testing, and incrementing clauses) can replace a
> while loop (with preceding initialization statement and included increment
> statement).  For example, mime_header() can be recoded using the pointer
> technique and content_parse() can be recoded using the switch and pointer
> techniques.

I will definately look at what you are refering to, however, I cant
guaruntee Ill
have time or the need to implement these changes.

> 
> David

Thanks for your input.  Input is very appreciated :)

-- 
mailto:[email protected]
Matt Brookings - Chief Technical Officer
Inter7 Internet Technologies, Inc.
www.inter7.com - 847-492-0470
Prices at http://www.inter7.com/prices