Your message dated Sun, 16 Dec 2018 14:46:12 +0000
with message-id <[email protected]>
and subject line Bug#836432: fixed in muse 3.0.2+ds1-1
has caused the Debian Bug report #836432,
regarding muse: Type problem with previously submitted patch
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
836432: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=836432
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: muse
Version: 2.1.2-2
Severity: normal

Hello,

In bug #831147 I submitted a patch, which was incorporated as
"1002-abs-errors.patch". However, there is a problem with the changes to
muse/midiedit/ecanvas.cpp

When <math.h> is included, it seems that GCC 6 provides the C++
overloads of the functions in the global namespace. In this case,
floating-point overloads of abs() are provided.

The arguments to the invocations of abs() in ecanvas.cpp are unsigned
int, and it turns out that they actually call the "double abs(double)"
overload, rather than an integer overload as intended.

You can verify this with a simple test program:

#include <iostream>
#include <math.h>
#include <typeinfo>

int main() {
        unsigned int x = 0;
        std::cout << typeid(abs(x)).name() << std::endl;
        return 0;
}

Running the output of this program through "c++filt -t" gives "double".

I don't think this is a major problem, but I think it should be fixed. I
will send an updated version of the patch that references this bug report.

Philip Chung

--- End Message ---
--- Begin Message ---
Source: muse
Source-Version: 3.0.2+ds1-1

We believe that the bug you reported is fixed in the latest version of
muse, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Sebastian Ramacher <[email protected]> (supplier of updated muse package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Sun, 16 Dec 2018 14:37:01 +0100
Source: muse
Binary: muse
Architecture: source
Version: 3.0.2+ds1-1
Distribution: unstable
Urgency: medium
Maintainer: Debian Multimedia Maintainers <[email protected]>
Changed-By: Sebastian Ramacher <[email protected]>
Description:
 muse       - Qt-based audio/MIDI sequencer
Closes: 836432 853565 875049 887794
Changes:
 muse (3.0.2+ds1-1) unstable; urgency=medium
 .
   * Team upload.
 .
   [ Ondřej Nový ]
   * d/copyright: Use https protocol in Format field
   * d/control: Set Vcs-* to salsa.debian.org
   * d/changelog: Remove trailing whitespaces
   * d/watch: Use https protocol
 .
   [ Felipe Sateler ]
   * Change maintainer address to [email protected]
 .
   [ Sebastian Ramacher ]
   * New upstream release. (Closes: #887794)
     - Fix build with GCC 7. (Closes: #853565)
     - Switch to Qt5. (Closes: #875049)
   * debian/watch: Update watch file for 3.0.x and handle repacking.
   * debian/copyright:
     - Remove embedded lilv, lv2, serd, sord and sratom.
     - Drop ladspa.h paragraph.
   * debian/patches:
     - Refresh patches.
     - Apply upstream patches to fix build with Qt 5.11.
     - Drop 1002-abs-errors.patch. The issue was fixed upstream. (Closes:
       #836432)
   * debian/: Bump debhelper compat to 11.
   * debian/control:
     - Also depend on python3 for some scripts.
     - Update build dependencies based on new requirements.
     - Bump Standards-Version.
   * debian/rules: Do not override CMake build type. The default does the right
     thing.
Checksums-Sha1:
 94a0bac196124a34bb7a71318342e763e185dc63 2201 muse_3.0.2+ds1-1.dsc
 e64e828c4efb9325112cc75d31575ddec623a0d8 5934820 muse_3.0.2+ds1.orig.tar.xz
 10a730ae9ef28fe2da390d9b4ee62221393f3783 29000 muse_3.0.2+ds1-1.debian.tar.xz
Checksums-Sha256:
 620d556b5054ad2363eb42fcd5208ed6120ee468dac1e6e0d9c795d0fb35261d 2201 
muse_3.0.2+ds1-1.dsc
 2a99f55c93d9bb836e61ddb77fd936e1f7de8d7ecbb73e7bde37518dea0dc4ba 5934820 
muse_3.0.2+ds1.orig.tar.xz
 9ca9f3744c7fc6cdd827b934bc0a5953a7701fb778d9cc3ca27c4a66ee70e5e5 29000 
muse_3.0.2+ds1-1.debian.tar.xz
Files:
 198d28319a1e6bb2bc486464a524ddef 2201 sound optional muse_3.0.2+ds1-1.dsc
 bf804010f37d6b980d9df860b55d83ae 5934820 sound optional 
muse_3.0.2+ds1.orig.tar.xz
 e48b8022a6ae9d630028f8199413d8bd 29000 sound optional 
muse_3.0.2+ds1-1.debian.tar.xz

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEE94y6B4F7sUmhHTOQafL8UW6nGZMFAlwWWMcACgkQafL8UW6n
GZO6QQ//cawhEYE/v3dQPMekepmOiQhNcwanfxmte3lvxh+48uXdddIpak0nUX2s
aS1P7c4JOcgzXgESbqNgzis/7IWyj7N48IXggoMT67EbcaBGGGSDPvIwmSQrJnvl
4dDKr/PeSaOV6GfTKWzhEGekuZpNHUZwLxPsPdNtDJmisxyF16ly8tPh5af12/sV
mleyDj0otEzW4w0YDJZClSEmQaUcQCiBdGrL978eW33Up4p7QlAyIrUvn7+IozbS
r35s6+KVDtcXFmJh0aTtE+/aaZ8M4MWMwiX1ODrV0Mg4gEMnDs+6EVUYRpvuL8mg
IMwXFE3olcsYABRUYOm9BY+uoHDb+mJl6Ul9e/uUsHrUvDaNHVhq1ycJ8rRTMDOC
k3Op/xATqtMaQ5KPkvAdFUUKLxoJmxnyqKguKcHDC9ONsMKshy2XG7JfOovR+EC4
7s4SUDHb6fZ7uOpZWJnM92IRWwEWpOv6Yw5UIRdC3NXNLeIaacN/3o5ByDQ+X9C5
tCqwOi9uvMj75IhGvlYDXjdPKq0mhY42jyLg4NIGx7anBCCL0Hdx5xVZZDWHWWVS
FRhyMUnmufnHjkYPluVfnbsI6Y6hZWsHZcyQWnyg7NzQjkOnGETDPPi0UkVWoD0q
cnil5N9/Rq0ipulHNREHyE4Zuy+ypqCYimjkAV/m3Rv55M6DbYg=
=rmZ8
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to