[med-svn] [Git][med-team/libedlib][master] 2 commits: Declare characters signed explicitly

[Andreas Tille]

Andreas Tille pushed to branch master at Debian Med / libedlib


Commits:
3710a7d4 by Andreas Tille at 2018-12-19T11:49:48Z
Declare characters signed explicitly

- - - - -
03c4d665 by Andreas Tille at 2018-12-19T11:52:07Z
Upload to unstable

- - - - -


3 changed files:

- debian/changelog
- debian/patches/series
- + debian/patches/signed_char.patch


Changes:

=====================================
debian/changelog
=====================================
@@ -1,3 +1,10 @@
+libedlib (1.2.3-5) unstable; urgency=medium
+
+  * Declare characters signed explicitly
+    Closes: #911967
+
+ -- Andreas Tille <tille at debian.org>  Wed, 19 Dec 2018 12:50:07 +0100
+
 libedlib (1.2.3-4) unstable; urgency=medium
 
   * Make sure the executable is found in test


=====================================
debian/patches/series
=====================================
@@ -1,3 +1,4 @@
 soversion.patch
 do_not_build_hello_example.patch
 cython3.patch
+signed_char.patch


=====================================
debian/patches/signed_char.patch
=====================================
@@ -0,0 +1,26 @@
+Description: Declare characters signed explicitly
+Bug-Debian: https://bugs.debian.org/911967
+Origin: https://stackoverflow.com/questions/31634962/narrowing-conversion-from-int-to-char-inside-for-legal-values-when-cro
+Author: Andreas Tille <tille at debian.org>
+Last-Update: Wed, 19 Dec 2018 12:09:52 +0100
+
+--- a/test/runTests.cpp
++++ b/test/runTests.cpp
+@@ -412,12 +412,12 @@ bool test10() {
+ bool test11() {
+     int queryLength = 8;
+     int targetLength = 8;
+-    char query[8] =  {-127, 127, -55, 0, 42, 0,      127, -55};
+-    char target[8] = {-127, 127,      0, 42, 0, -55, 127,  42};
++    signed char query[8] =  {-127, 127, -55, 0, 42, 0,      127, -55};
++    signed char target[8] = {-127, 127,      0, 42, 0, -55, 127,  42};
+ 
+-    bool r = executeTest(query, queryLength, target, targetLength, EDLIB_MODE_HW);
+-    r = r && executeTest(query, queryLength, target, targetLength, EDLIB_MODE_NW);
+-    r = r && executeTest(query, queryLength, target, targetLength, EDLIB_MODE_SHW);
++    bool r = executeTest((char *)query, queryLength, (char *)target, targetLength, EDLIB_MODE_HW);
++    r = r && executeTest((char *)query, queryLength, (char *)target, targetLength, EDLIB_MODE_NW);
++    r = r && executeTest((char *)query, queryLength, (char *)target, targetLength, EDLIB_MODE_SHW);
+     return r;
+ }
+ 



View it on GitLab: https://salsa.debian.org/med-team/libedlib/compare/e27bd1da2e088f52a42007831b4de3f8b9fe27d6...03c4d665440a1fc6b7b019102e2e0d24972afc14

-- 
View it on GitLab: https://salsa.debian.org/med-team/libedlib/compare/e27bd1da2e088f52a42007831b4de3f8b9fe27d6...03c4d665440a1fc6b7b019102e2e0d24972afc14
You're receiving this email because of your account on salsa.debian.org.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/debian-med-commit/attachments/20181219/55694ded/attachment-0001.html>