On Tue, Jun 27, 2017 at 10:19 PM, Dave Young <dyoung at redhat.com> wrote: > On 06/19/17 at 03:23pm, Cong Wang wrote: >> - >> - /* Move to next record */ >> - current_idx = log_next(buf, current_idx); >> + free(msg); >> + /* length == 0 indicates the end of the buffer; wrap */ > > Since the code is restructured, above line can be dropped, other wise: Yeah, I will improve that comment and send v2. Thanks!
