summaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorAndreas Fenkart <andreas.fenkart@digitalstrom.com>2016-03-11 09:39:38 +0100
committerTom Rini <trini@konsulko.com>2016-03-26 18:49:29 -0400
commit69067a34b1ab1fb14f0183acb85358db60aac484 (patch)
tree4c252a4c95244a1664cc525562e5e7f1069ae17d /tools
parent938c29ff41b40a1b6cafc9bcc81b89ad2bd537ba (diff)
downloadtalos-obmc-uboot-69067a34b1ab1fb14f0183acb85358db60aac484.tar.gz
talos-obmc-uboot-69067a34b1ab1fb14f0183acb85358db60aac484.zip
tools: env: fw_parse_script: simplify removal of newline/carriage return
fgets returns when the first '\n' is found Signed-off-by: Andreas Fenkart <andreas.fenkart@digitalstrom.com>
Diffstat (limited to 'tools')
-rw-r--r--tools/env/fw_env.c10
1 files changed, 4 insertions, 6 deletions
diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c
index 60574f249d..5c7505c0fd 100644
--- a/tools/env/fw_env.c
+++ b/tools/env/fw_env.c
@@ -566,14 +566,12 @@ int fw_parse_script(char *fname)
}
/* Drop ending line feed / carriage return */
- while (len > 0 && (dump[len - 1] == '\n' ||
- dump[len - 1] == '\r')) {
- dump[len - 1] = '\0';
- len--;
- }
+ dump[--len] = '\0';
+ if (len && dump[len - 1] == '\r')
+ dump[--len] = '\0';
/* Skip comment or empty lines */
- if ((len == 0) || dump[0] == '#')
+ if (len == 0 || dump[0] == '#')
continue;
/*
OpenPOWER on IntegriCloud