]> code.ossystems Code Review - openembedded-core.git/commitdiff
libpfm4: fix compile error
authorKai Kang <kai.kang@windriver.com>
Wed, 29 Jun 2016 08:08:45 +0000 (16:08 +0800)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 8 Jul 2016 08:55:40 +0000 (09:55 +0100)
It fails to compile libpfm4 with misleading-indentation error:

| syst_count.c:346:3: error: this 'for' clause does not guard...  [-Werror=misleading-indentation]
|   for(c=cmin ; c < cmax; c++)
|      ^~~

Add clauses to fix it.

Signed-off-by: Kai Kang <kai.kang@windriver.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
meta/recipes-kernel/libpfm/files/fix-misleading-indentation-error.patch [new file with mode: 0644]
meta/recipes-kernel/libpfm/libpfm4_4.6.0.bb

diff --git a/meta/recipes-kernel/libpfm/files/fix-misleading-indentation-error.patch b/meta/recipes-kernel/libpfm/files/fix-misleading-indentation-error.patch
new file mode 100644 (file)
index 0000000..25d47ef
--- /dev/null
@@ -0,0 +1,38 @@
+Upstream-Status: Pending
+
+Fix compile error:
+
+| syst_count.c:346:3: error: this 'for' clause does not guard... [-Werror=misleading-indentation] 
+|    for(c=cmin ; c < cmax; c++) 
+|       ^~~ 
+
+Signed-off-by: Kai Kang <kai.kang@windriver.com>
+---
+diff --git a/perf_examples/syst_count.c b/perf_examples/syst_count.c
+index 0d53078..2dfb9b9 100644
+--- a/perf_examples/syst_count.c
++++ b/perf_examples/syst_count.c
+@@ -343,8 +343,9 @@ measure(void)
+               for (delay = 1 ; delay <= options.delay; delay++) {
+-              for(c=cmin ; c < cmax; c++)
+-                      start_cpu(c);
++                      for(c=cmin ; c < cmax; c++) {
++                              start_cpu(c);
++                      }
+                       if (0) {
+                               tv.tv_sec = 0;
+@@ -353,8 +354,9 @@ measure(void)
+                       } else
+                               sleep(1);
+-              for(c=cmin ; c < cmax; c++)
+-                      stop_cpu(c);
++                      for(c=cmin ; c < cmax; c++) {
++                              stop_cpu(c);
++                      }
+                       for(c = cmin; c < cmax; c++) {
+                               printf("# %'ds -----\n", delay);
index 0b024ad61dcfccf98eb261d134c4d57bf720da28..4136eac81f794c97c4dcadffad1c18f35439f864 100644 (file)
@@ -14,6 +14,7 @@ COMPATIBLE_HOST = "powerpc64"
 
 SRC_URI = "http://downloads.sourceforge.net/project/perfmon2/${BPN}/libpfm-${PV}.tar.gz \
            file://0001-Makefile-Add-LDFLAGS-variable-to-SLDFLAGS.patch \
+           file://fix-misleading-indentation-error.patch \
           "
 
 SRC_URI[md5sum] = "5077b9022440e4951d96f2d0e73bd487"