Bug#836432: muse: Type problem with previously submitted patch

Philip Chung philipchung1995 at yahoo.com
Sat Sep 3 03:49:28 UTC 2016


Control: tags -1 patch

On Fri, 2 Sep 2016 20:39:37 -0700 Philip Chung
<philipchung1995 at yahoo.com> wrote:
> 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
> 
> [snip]
>
> 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.

Attached is the updated patch. It casts the arguments to int before
calling abs() so that the proper overload is used. Also, the include of
<math.h> is changed to <stdlib.h> (which is where the integer versions
of abs() normally reside).

Philip Chung
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 1002-abs-errors.patch
Type: text/x-patch
Size: 3398 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20160902/66e7e117/attachment.bin>


More information about the pkg-multimedia-maintainers mailing list