[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gap-dev-discuss] Fixing Cynthiune Timidity Bundle

From: Riccardo Mottola
Subject: Re: [Gap-dev-discuss] Fixing Cynthiune Timidity Bundle
Date: Sun, 13 May 2012 20:56:39 +0200
User-agent: Mozilla/5.0 (X11; NetBSD i386; rv:12.0) Gecko/20120510 Thunderbird/12.0


I don't know how to handle the executable stuff, but your buffer patch seems sane, thus commit it!


On 05/13/12 15:12, Sebastian Reitenbach wrote:

I thought I test some more file types, and chose a MIDI file.
First, I recognized, when having the Timidity bundles installed,
but not timidity itself, Cynthiune will crash weirdly, when you try
to add a .mid file to the playlist.

Then I found it still crashing, looking at the Timidity.m, I found the
path to timidity hardcoded to /usr/bin/timidity. But on OpenBSD, its
installed in /usr/local/bin. So changed that #define to not being
a full qualified path. That enabled me to add a .mid file to the

Then trying to play that .mid file, Cynthiune crashed again :(
Looking closer, I found the Timidity Bundle was trying to copy a buffer
of size 65535 into a smaller buffer of DEFAULT_BUFFER_SIZE.
Therefore #import<Cynthiune/Output.h>  and use the DEFAULT_BUFFER_SIZE.

Now Cynthiune happily plays the .mid file for me on i386.

Since cynthiune crashes, when no timidity binary is found, do we
maybe want to disable Timidity plugin by default?

Or, I know there is a method to find a binary in $PATH, which in turn
returns the full qualified file name. Maybe we could check in -init
if the timidity binary can be found along the path, and if not, return nil
in -init?


Index: Timidity.m
RCS file: /sources/gap/gap/user-apps/Cynthiune/Bundles/Timidity/Timidity.m,v
retrieving revision
diff -u -r1.1.1.1 Timidity.m
--- Timidity.m  25 Mar 2012 07:48:23 -0000
+++ Timidity.m  13 May 2012 12:59:48 -0000
@@ -24,13 +24,14 @@


  #import "Timidity.h"

  #define LOCALIZED(X) _b ([Timidity class], X)

-#define timidity "/usr/bin/timidity"
+#define timidity "timidity"
  #define rate 22050

  @implementation Timidity : NSObject
@@ -175,7 +176,7 @@
    NSData *bytes;
    int size;

-  bytes = [in readDataOfLength: 65536];
+  bytes = [in readDataOfLength: DEFAULT_BUFFER_SIZE];
    if (bytes)
        size = [bytes length];

reply via email to

[Prev in Thread] Current Thread [Next in Thread]