From: Khem Raj <raj.khem@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: [meta-oe][PATCH 04/16] libplist: Squash warnings found by clang
Date: Tue, 18 Oct 2016 22:38:22 -0700 [thread overview]
Message-ID: <20161019053834.11523-4-raj.khem@gmail.com> (raw)
In-Reply-To: <20161019053834.11523-1-raj.khem@gmail.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
.../files/0001-Fix-warnings-found-with-clang.patch | 51 ++++++++++++++++++++++
.../libimobiledevice/libplist_1.8.bb | 3 +-
2 files changed, 53 insertions(+), 1 deletion(-)
create mode 100644 meta-oe/recipes-extended/libimobiledevice/files/0001-Fix-warnings-found-with-clang.patch
diff --git a/meta-oe/recipes-extended/libimobiledevice/files/0001-Fix-warnings-found-with-clang.patch b/meta-oe/recipes-extended/libimobiledevice/files/0001-Fix-warnings-found-with-clang.patch
new file mode 100644
index 0000000..f8114a0
--- /dev/null
+++ b/meta-oe/recipes-extended/libimobiledevice/files/0001-Fix-warnings-found-with-clang.patch
@@ -0,0 +1,51 @@
+From ad1be542b87b3186f8ef7bee2c594daefe5bb4c8 Mon Sep 17 00:00:00 2001
+From: Khem Raj <raj.khem@gmail.com>
+Date: Tue, 18 Oct 2016 21:31:40 +0000
+Subject: [PATCH] Fix warnings found with clang
+
+| /mnt/oe/openembedded-core/build/workspace/sources/libplist/src/base64.c:107:12: error: non-void function 'base64decode' should return a value [-Wreturn-type]
+| if (!buf) return;
+| ^
+| /mnt/oe/openembedded-core/build/workspace/sources/libplist/src/base64.c:109:16: error: non-void function 'base64decode' should return a value [-Wreturn-type]
+| if (len <= 0) return;
+
+Upstream-Status: Pending
+
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+---
+ libcnary/node.c | 2 +-
+ src/base64.c | 4 ++--
+ 2 files changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/libcnary/node.c b/libcnary/node.c
+index 1f9f669..d6f3f63 100644
+--- a/libcnary/node.c
++++ b/libcnary/node.c
+@@ -104,7 +104,7 @@ int node_detach(node_t* parent, node_t* child) {
+
+ int node_insert(node_t* parent, unsigned int index, node_t* child)
+ {
+- if (!parent || !child) return;
++ if (!parent || !child) return -1;
+ child->isLeaf = TRUE;
+ child->isRoot = FALSE;
+ child->parent = parent;
+diff --git a/src/base64.c b/src/base64.c
+index 65c6061..531a06a 100644
+--- a/src/base64.c
++++ b/src/base64.c
+@@ -104,9 +104,9 @@ static int base64decode_block(unsigned char *target, const char *data, size_t da
+
+ unsigned char *base64decode(const char *buf, size_t *size)
+ {
+- if (!buf) return;
++ if (!buf) return 0;
+ size_t len = strlen(buf);
+- if (len <= 0) return;
++ if (len <= 0) return 0;
+ unsigned char *outbuf = (unsigned char*)malloc((len/4)*3+3);
+
+ unsigned char *line;
+--
+1.9.1
+
diff --git a/meta-oe/recipes-extended/libimobiledevice/libplist_1.8.bb b/meta-oe/recipes-extended/libimobiledevice/libplist_1.8.bb
index 6e8d69f..10d40fb 100644
--- a/meta-oe/recipes-extended/libimobiledevice/libplist_1.8.bb
+++ b/meta-oe/recipes-extended/libimobiledevice/libplist_1.8.bb
@@ -10,7 +10,8 @@ inherit cmake pkgconfig
SRC_URI = "http://www.libimobiledevice.org/downloads/libplist-${PV}.tar.bz2 \
file://fix-parallel-make.patch \
-"
+ file://0001-Fix-warnings-found-with-clang.patch \
+ "
SRC_URI[md5sum] = "2a9e0258847d50f9760dc3ece25f4dc6"
SRC_URI[sha256sum] = "a418da3880308199b74766deef2a760a9b169b81a868a6a9032f7614e20500ec"
--
2.10.0
next prev parent reply other threads:[~2016-10-19 5:38 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-19 5:38 [meta-oe][PATCH 01/16] boinc: Add recipe for boinc-client Khem Raj
2016-10-19 5:38 ` [meta-gnome][PATCH 02/16] libbonobo: Fix missing dep on orbit2-native and build with clang Khem Raj
2016-10-19 20:09 ` akuster808
2016-10-20 2:45 ` Khem Raj
2016-10-19 5:38 ` [meta-oe][PATCH 03/16] libmbim: Fix " Khem Raj
2016-10-19 5:38 ` Khem Raj [this message]
2016-10-19 5:38 ` [meta-oe][PATCH 05/16] frame: " Khem Raj
2016-10-19 5:38 ` [meta-python][PATCH 06/16] python-greenlet: Rename register from rX to xX for aarch64 Khem Raj
2016-10-19 5:38 ` [meta-multimedia][PATCH 07/16] libdc1394: Upgrade to 2.2.4 Khem Raj
2016-10-19 5:38 ` [meta-networking][PATCH 08/16] lowpan-tools: Fix errors found with clang Khem Raj
2016-10-19 5:38 ` [meta-oe][PATCH 09/16] gpm: Remove nested functions Khem Raj
2016-10-19 5:38 ` [meta-oe][PATCH 10/16] flashrom: Fix build with clang and aarch64 Khem Raj
2016-10-19 5:38 ` [meta-networking][PATCH 11/16] openl2tp: Fix build with clang Khem Raj
2016-10-19 5:38 ` [meta-oe][PATCH 12/16] glcompbench: Demand c++11 explicitly Khem Raj
2016-10-19 5:38 ` [meta-oe][PATCH 13/16] joe: Fix build with clang Khem Raj
2016-10-19 5:38 ` [meta-oe][PATCH 14/16] fwts: Update to 16.09 release Khem Raj
2016-10-25 13:29 ` Martin Jansa
2016-10-19 5:38 ` [meta-networking][PATCH 15/16] libmnl, nftables: Update versions Khem Raj
2016-10-19 5:38 ` [meta-oe][PATCH 16/16] mg: Update to 20161005 Khem Raj
2016-10-19 20:06 ` [meta-oe][PATCH 01/16] boinc: Add recipe for boinc-client akuster808
2016-10-25 13:32 ` Martin Jansa
2016-10-25 13:49 ` Khem Raj
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161019053834.11523-4-raj.khem@gmail.com \
--to=raj.khem@gmail.com \
--cc=openembedded-devel@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.