[Crash-utility] [PATCH V3] Add -C option for search

Dave Anderson anderson at redhat.com
Thu Feb 16 16:56:39 UTC 2012



----- Original Message -----
> Hello Dave,
> 
> The new patch is attached.
> 
> I  simplified the display_with_pre_and_post() function by calling the
> currently-existing display_memory() function, and made the output
> readable according to your advice.
> 
> Thanks
> Zhang Yanfei
> 

It's looking pretty good -- a couple minor points/questions:

(1) in display_memory(), why re-initialize the error_handle each
    time through the loop, instead of doing it just once?
(2) in display_memory(), shouldn't the "lost" variable be reset 
    to 0 after a line of output is printed?

Dave




More information about the Crash-utility mailing list