>>>>> On Mon, 10 Jan 2005 15:02:29 +0000, Dave Shield <[EMAIL PROTECTED]> said:

Dave> I'd say the current documentation is fairly vague about how to
Dave> use it.  That's partly why I'm writing the book :-)

Hey!  I only write vague documentation, you should know that.  At
least there *is* documentation.  Back in my day, I had to learn C by
dissecting a compiler to see what it did!  I didn't even know it was a
compiler!  It was just called "a.out" and I figured the rest out just
from looking at it!  Ok, it's late.  Don't read mail when tired...

Dave> Secondly, I've tweaked the stash-cache handler to automatically
Dave> inject the new stash-to-next handler into the chain if it
Dave> detects that this is needed.
Dave> Can you suggest a better way?
>> 
>> Depends on where you do it.  Neither you nor Robert clarified
>> what you meant by "active".

Dave> During the course of processing a request.

Err...  um, *why* would you ever do it during request processing?
That definitely doesn't seem like a reasonable place to do that
checking.  Why not parallel the way the bulk_to_next handler does it?
(which admittedly, I think is higher in the agent code if memory
serves and thus doesn't feel as flexible, but sure feels safer)

Dave> a)  Live dangerously, and inject the stash_to_next handler
Dave> into the chain while it's processing the (first) request,
Dave> [ The current mode of operation ]

not in my favor, as mentioned.  Also sacrifices efficiency during
processing of the important stuff.

Dave> b)  Have 'netsnmp_get_stash_cache_handler()' return a
Dave> mini-chain of *both* the stash_cache and stash_to_next
Dave> handlers, and inject these two into the handler chain

that'd be acceptable in my eyes.

Dave> [ assuming 'netsnmp_inject_handler' supports
Dave> working with more than one handler, of course ]

But that'd need to be fixed, as I don't think it does.

Dave> c)  Merge 'stash_to_next' into the main stash_cache helper

nah.

Dave> Oh, and I probably ought to include a fourth:

Dave> d)  Forget about extending stash_cache support altogether.

Nah.

e) agent_handler.c near line 240

-- 
Wes Hardaker
Sparta


-------------------------------------------------------
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
_______________________________________________
Net-snmp-coders mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to