msrtool: Remove indent by using continue inside for() to avoid an if block
authorPeter Stuge <peter@stuge.se>
Sat, 16 Jan 2010 17:21:17 +0000 (17:21 +0000)
committerPeter Stuge <peter@stuge.se>
Sat, 16 Jan 2010 17:21:17 +0000 (17:21 +0000)
The only actual code change is from
if (.. >= 1) {
}
to
if (.. < 1)
continue
so this is pretty trivial.

Signed-off-by: Peter Stuge <peter@stuge.se>
Acked-by: Peter Stuge <peter@stuge.se>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5020 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

util/msrtool/msrtool.c

index 56d2591db4161a4b7b76a24d4cc345bd6bfafdd9..7ef40d82b4465230b1075b76dcfca5cd5a63b043 100644 (file)
@@ -182,7 +182,7 @@ done:
 int do_diff(const char *difffn) {
        char tmpfn[20], line[512];
        size_t start, len;
-       int ret = 1, tmp;
+       int ret = 1, found, tmp;
        FILE *fin = NULL, *fout = stdout;
        uint8_t rev = 0;
        uint32_t addr, linenum;
@@ -203,19 +203,20 @@ int do_diff(const char *difffn) {
                goto done;
        for (linenum = 1; NULL != fgets(line, sizeof(line), fin); ++linenum) {
                start = (0 == strncmp("0x", line, 2)) ? 2 : 0;
-               if (sscanf(line + start, "%8x %n%*x", &addr, &tmp) >= 1) {
-                       start += tmp;
-                       for (len = strlen(line) - 1; NULL != strchr("\r\n", line[len]); --len)
-                               line[len] = 0;
-                       if (!str2msr(line + start, &mf)) {
-                               fprintf(stderr, "%s:%d: invalid MSR value '%s'\n", difffn, linenum, line + start);
-                               continue;
-                       }
-                       if (!sys->rdmsr(cpu, addr, &mhw))
-                               goto done;
-                       if (diff_msr(fout, addr, rev ? mhw : mf, rev ? mf : mhw))
-                               fprintf(fout, "\n");
+               found = sscanf(line + start, "%8x %n%*x", &addr, &tmp);
+               if (found < 1)
+                       continue;
+               start += tmp;
+               for (len = strlen(line) - 1; NULL != strchr("\r\n", line[len]); --len)
+                       line[len] = 0;
+               if (!str2msr(line + start, &mf)) {
+                       fprintf(stderr, "%s:%d: invalid MSR value '%s'\n", difffn, linenum, line + start);
+                       continue;
                }
+               if (!sys->rdmsr(cpu, addr, &mhw))
+                       goto done;
+               if (diff_msr(fout, addr, rev ? mhw : mf, rev ? mf : mhw))
+                       fprintf(fout, "\n");
        }
        if (!feof(fin))
                fprintf(stderr, "%s:%d: fgets: %s\n", difffn, linenum, strerror(errno));