Code review comment for lp://qastaging/~jdstrand/snap-confine/preprocessor

Revision history for this message
Tyler Hicks (tyhicks) wrote :

Thanks for the explanation. That makes sense.

Note that the sizeof() in preprocess_filter() is fine since the buf array is declared in scope.

Feel free to merge after you've adjusted read_line() to have a 'size_t buf_len' argument.

« Back to merge proposal