Bruce> It's a one-liner in mib.c.  It effects everybody.
Bruce> *(*buf + (*out_len)++) = '\n';

Bruce> Comment that line out and everything appears to be fine (at least on
Bruce> Solaris).

Have you tried this with "long" hex strings (i.e. over 16 octets, so they
are displayed split over two or more lines) ?


Bruce> When did this get added and why?
Wes>   revision 5.77


Bruce> Nope, it's a lot older than that.  
Bruce> There was a line that said: 
Bruce>          if (len > 16) { *buf++ = '\n'; *buf = 0; }
Bruce> part of sprint_hexstring.

But that only added an *internal* newline character - it wouldn't have been
triggered the last time through, when len <= 16.

I suspect that _sprint_hexstring_line() may need an extra flag parameter to
indicate whether or not to add this newline character.


But I also agree with Wes:

Wes>  It's not critical and thus can't be fixed for 5.2 

Dave



-------------------------------------------------------
This SF.Net email is sponsored by:
Sybase ASE Linux Express Edition - download now for FREE
LinuxWorld Reader's Choice Award Winner for best database on Linux.
http://ads.osdn.com/?ad_id=5588&alloc_id=12065&op=click
_______________________________________________
Net-snmp-coders mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to