aboutsummaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorElliott Hughes <enh@google.com>2019-07-19 09:48:04 -0700
committerRob Landley <rob@landley.net>2019-07-22 16:42:07 -0500
commitb30674681b9d72430a029ba7bd3e16a3139244f0 (patch)
treef4b9b66e5c1537ea2a73d606f6b8aa3fe9e102c2 /scripts
parent43d398ad5d7b14fb344fc2e5338177761b9a199a (diff)
downloadtoybox-b30674681b9d72430a029ba7bd3e16a3139244f0.tar.gz
Start replacing get_line() with getline().
I started this last night, but thought I'd aim to send multiple small patches rather than work through all the callers and send one big patch. I've deliberately chosen the ugly name `allocated_length` because we've had historical bugs where folks think this a line length in the sense of the return value. I do wonder whether we should actually have some kind of getline() wrapper that hides the `char *`/`size_t` pair in lib/, which makes the function easier to use in most cases but does add the less common gotcha that you wouldn't be able to getline() through multiple files at once (which does happen in at least one toy). But maybe the real fix is to look harder for places where we can just use loopfiles_lines? Speaking of which, should we actually add two more arguments to that? Specifically: switch it to getdelim() rather than getline() behind the scenes, and also add a way to have the trailing '\n' automatically removed, since that seems to be what most callers want? Anyway, that seemed like enough questions that it was time to send this initial patch out before doing too much more...
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions