Skip to content

Commit

Permalink
tweaks: store a result, to avoid calling a function twice
Browse files Browse the repository at this point in the history
  • Loading branch information
Benno Schulenberg committed Feb 18, 2022
1 parent 1b186fc commit 756cd4a
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/text.c
Original file line number Diff line number Diff line change
Expand Up @@ -2649,10 +2649,11 @@ void do_linter(void)
if (onelint != pointer) {
char *filename, *linestring, *colstring;
char *complaint = copy_of(onelint);
char *spacer = strstr(complaint, " ");

/* The recognized format is "filename:line:column: message",
* where ":column" may be absent or be ",column" instead. */
if ((filename = strtok(onelint, ":")) && strstr(complaint, " ")) {
if ((filename = strtok(onelint, ":")) && spacer) {
if ((linestring = strtok(NULL, ":"))) {
if ((colstring = strtok(NULL, " "))) {
ssize_t linenumber = strtol(linestring, NULL, 10);
Expand Down Expand Up @@ -2681,7 +2682,7 @@ void do_linter(void)
curlint->filename = copy_of(filename);
curlint->lineno = linenumber;
curlint->colno = colnumber;
curlint->msg = copy_of(strstr(complaint, " ") + 1);
curlint->msg = copy_of(spacer + 1);

if (lints == NULL)
lints = curlint;
Expand Down

0 comments on commit 756cd4a

Please sign in to comment.