excluding characters

fgets stops when it sees newline and stores it, so I think you need simply something like this:

Code:

size_t len;
while(fgets(buffer, sizeof buffer, f)){
    len = strlen(buffer);
    if(buffer[len-1] == '\n')
        bufferl[--len] = '\0';
    ...
}

Originally Posted by mzn

fgets stops when it sees newline and stores it, so I think you need simply something like this:

Yes, that can work, but it is simpler to use strchr instead of strlen.

Originally Posted by laserlight

Yes, that can work, but it is simpler to use strchr instead of strlen.

I don't think he gets away any easier using strchr. But what I do think is that strlen return value just might be more of use later in his code, thats only my guess though

Originally Posted by mzn

I don't think he gets away any easier using strchr.

It is less work, both in the code and in what you have to consider, i.e., whether array out of bounds access is possible. For example:

Code:

while (fgets(buffer, sizeof buffer, f)) {
    char *p = strchr(buffer, '\n');
    if (p)
        *p = '\0';
    /* ... */
}

Originally Posted by mzn

But what I do think is that strlen return value just might be more of use later in his code, thats only my guess though

Both are true: the length of the string may be useful, but it may also be unnecessary.