[nethack] 02/04: Fix permanently identify bug on arm* ports. (Closes: #828625)

[James Cowgill]

This is an automated email from the git hooks/post-receive script.

jcowgill pushed a commit to branch master
in repository nethack.

commit 6cf5d0d63b9d52241254a00052592920336b784f
Author: Mo Jun <royclark086 at gmail.com>
Date:   Sun Jun 26 15:17:16 2016 +0100

    Fix permanently identify bug on arm* ports. (Closes: #828625)
---
 debian/patches/0026-fix-permanently-identify.patch | 19 +++++++++++++++++++
 debian/patches/series                              |  1 +
 2 files changed, 20 insertions(+)

diff --git a/debian/patches/0026-fix-permanently-identify.patch b/debian/patches/0026-fix-permanently-identify.patch
new file mode 100644
index 0000000..12d2afe
--- /dev/null
+++ b/debian/patches/0026-fix-permanently-identify.patch
@@ -0,0 +1,19 @@
+Description: fix permanently identify bug
+ The return value of display_inventory() is from a type char value;
+ since the type char is unsigned on port arm by default, the if statement will
+ never success and identify will not be commited.
+Author: Mo Jun <royclark086 at gmail.com>
+Bug-Debian: https://bugs.debian.org/828625
+Forwarded: yes
+Reviewed-By: James Cowgill <jcowgill at debian.org>
+--- a/src/cmd.c
++++ b/src/cmd.c
+@@ -592,7 +592,7 @@
+ {
+     if (wizard) {
+         iflags.override_ID = (int) cmd_from_func(wiz_identify);
+-        if (display_inventory((char *) 0, TRUE) == -1)
++        if (display_inventory((char *) 0, TRUE) == (char) -1)
+             identify_pack(0, FALSE);
+         iflags.override_ID = 0;
+     } else
diff --git a/debian/patches/series b/debian/patches/series
index 8111482..d8c81ed 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -16,3 +16,4 @@ u1-avoid-crash-for-long-lines-in-nethackrc.patch
 0023-gcc-warning-fixes.patch
 0024-lisp-3.6.patch
 0025-reproducible-build.patch
+0026-fix-permanently-identify.patch

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-games/nethack.git