Rockbox mail archiveSubject: Re: cvs: apps settings.c,1.67,1.68
From: Magnus Holmgren (lear_at_algonet.se)
Björn Stenberg wrote:
> Modified Files:
Since I wrote the original patch, I just have to make a few comments. :)
The code was cleaned up a lot, which is nice, but the visual feedback and error handling is gone (and they caused much of the complexity). I'm not sure what is best though...
read_line was deliberately written to always null-terminate the buffer. That isn't the case any more, AFAICS, which could cause problems...
Page was last modified "Jan 10 2012" The Rockbox Crew