dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

Subject: Re: cvs: apps settings.c,1.67,1.68
From: Magnus Holmgren (
Date: 2002-10-04

Björn Stenberg wrote:

> Yes. I removed that, since I think it should rather be
> handled in tree.c after the loading is completed.

Though it's limited what can be handled there... :)

> It null terminates, unless the string was completely
> filled. This is standard behaviour, just like strncpy etc.
> However, I'm inclined to agree with you that this can cause
> problems, since users of read_line (such as myself ;) will
> likely forget to check this.

I dislike strncpy for that very reason.

Another read_line thing: a line consisting of nothing but a line feed returns 0, breaking the read... That's why I made read_line return true if something had been read.


Page was last modified "Jan 10 2012" The Rockbox Crew