Hello community,

here is the log from the commit of package awesfx for openSUSE:Factory checked 
in at 2019-09-07 11:50:28
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/awesfx (Old)
 and      /work/SRC/openSUSE:Factory/.awesfx.new.7948 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "awesfx"

Sat Sep  7 11:50:28 2019 rev:27 rq:728496 version:0.5.2

Changes:
--------
--- /work/SRC/openSUSE:Factory/awesfx/awesfx.changes    2019-04-04 
14:13:28.289329609 +0200
+++ /work/SRC/openSUSE:Factory/.awesfx.new.7948/awesfx.changes  2019-09-07 
11:50:31.706298792 +0200
@@ -1,0 +2,7 @@
+Thu Sep  5 14:10:09 CEST 2019 - [email protected]
+
+- Fix the build error and warning:
+  Fix-the-bogus-return-in-seq_set_gus_bank.patch
+  Fix-unused-variable-prev-in-strtoken.patch
+
+-------------------------------------------------------------------

New:
----
  Fix-the-bogus-return-in-seq_set_gus_bank.patch
  Fix-unused-variable-prev-in-strtoken.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ awesfx.spec ++++++
--- /var/tmp/diff_new_pack.sgOeyE/_old  2019-09-07 11:50:32.338298703 +0200
+++ /var/tmp/diff_new_pack.sgOeyE/_new  2019-09-07 11:50:32.342298702 +0200
@@ -12,7 +12,7 @@
 # license that conforms to the Open Source Definition (Version 1.9)
 # published by the Open Source Initiative.
 
-# Please submit bugfixes or comments via http://bugs.opensuse.org/
+# Please submit bugfixes or comments via https://bugs.opensuse.org/
 #
 
 
@@ -25,6 +25,8 @@
 Url:            https://github.com/tiwai/awesfx
 Source:         awesfx-%{version}.tar.gz
 Patch:          awesfx-0.5.1e-fix-bashisms.patch
+Patch1:         Fix-the-bogus-return-in-seq_set_gus_bank.patch
+Patch2:         Fix-unused-variable-prev-in-strtoken.patch
 BuildRequires:  alsa-devel
 BuildRequires:  libtool
 BuildRequires:  pkgconfig
@@ -40,6 +42,8 @@
 %prep
 %setup -q
 %patch -p1
+%patch1 -p1
+%patch2 -p1
 
 %build
 autoreconf -fi

++++++ Fix-the-bogus-return-in-seq_set_gus_bank.patch ++++++
>From 682d0caaffa9bfcc2381b947fb47395fb3da98b9 Mon Sep 17 00:00:00 2001
From: Takashi Iwai <[email protected]>
Date: Thu, 5 Sep 2019 14:06:55 +0200
Subject: [PATCH] Fix the bogus return in seq_set_gus_bank()

Fix the compile error:
 alsa.c: In function 'seq_set_gus_bank':
 alsa.c:149:9: error: 'return' with a value, in function returning void 
[-Werror=return-type]
   149 |  return snd_hwdep_ioctl(hwdep, SNDRV_EMUX_IOCTL_MISC_MODE, &mode);

Signed-off-by: Takashi Iwai <[email protected]>
---
 alsa.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/alsa.c b/alsa.c
index e5993cb344e2..b977f44695b6 100644
--- a/alsa.c
+++ b/alsa.c
@@ -146,5 +146,5 @@ void seq_set_gus_bank(int bank)
        mode.mode = AWE_MD_GUS_BANK;
        mode.value = bank;
        mode.value2 = 0;
-       return snd_hwdep_ioctl(hwdep, SNDRV_EMUX_IOCTL_MISC_MODE, &mode);
+       snd_hwdep_ioctl(hwdep, SNDRV_EMUX_IOCTL_MISC_MODE, &mode);
 }
-- 
2.16.4

++++++ Fix-unused-variable-prev-in-strtoken.patch ++++++
>From 0581458acc5f28ef50742805cf37278d979b1c12 Mon Sep 17 00:00:00 2001
From: Takashi Iwai <[email protected]>
Date: Thu, 5 Sep 2019 14:07:33 +0200
Subject: [PATCH] Fix unused variable prev in strtoken()

Signed-off-by: Takashi Iwai <[email protected]>
---
 awelib/bool.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/awelib/bool.c b/awelib/bool.c
index ebb4c6e6d65a..f89deb8a52f1 100644
--- a/awelib/bool.c
+++ b/awelib/bool.c
@@ -110,10 +110,10 @@ char *strtoken(char *src)
                                break;
                        vptr++;
                } else if (*vptr == '"' || *vptr == '\'') {
-                       int prev, quote;
-                       prev = quote = *vptr;
+                       int quote;
+                       quote = *vptr;
                        remove_letter(vptr);
-                       for (; *vptr; prev = *vptr, vptr++) {
+                       for (; *vptr; vptr++) {
                                if (*vptr == '\\') {
                                        remove_letter(vptr);
                                        if (*vptr)
-- 
2.16.4


Reply via email to