Fix Windows Driver

Add DriverCallback API.  The driver must call the DriverCallback API.
After changing, some MIDI sequencers will not freeze: XGWorks, Domino,
etc.

Signed-off-by: yta <yta@inter7.jp>

Fix ssize_t type test and conditional compilation

Fix ssize_t type test because older versions of autoconf do not
support AC_TYPE_SSIZE_T macro.  Also, fix conditional compilation for
older versions of MinGW.

Signed-off-by: TAMUKI Shoichi <tamuki@linet.gr.jp>

Revert some fix and add ssize_t type test.

Minor fixes for conditional compile.

Fix HTTPproxy not working on http URL.

Fix crash on sample size change while running.

Change Windows GUI volume slider max.

Volume control of W32GUI interface currently changes
rendering amplification, not playback volume.

Fix some compilation issues on Windows.

Update version number in Windows resource.

Fix tk interface compile with tk8.6

Direct access to interp->result is disallowed in 8.6, so use Tcl_GetStringResult instead. Also increase MAX_TK_MIDI_CHANNELS to fix compile warning.

Fix ao options crash

Initialize ao_options to NULL, so as to avoid crash when no options are specified.

Prefix ./ to escape_sjis.pl in Makefile to execute the script

Signed-off-by: TAMUKI Shoichi <tamuki@linet.gr.jp>

Gleaning various fixes

- Fix vwprintw to fit with newer PDCurses
- Fix control_mode_event args to use ptr_size_t
- Fix read_config_file to be used in another main
- Fix around MIDIOUTCAPS2 in windrv/timiditydrv.c

Signed-off-by: TAMUKI Shoichi <tamuki@linet.gr.jp>

Refactor w32_a.c and disable the debug messages

Signed-off-by: TAMUKI Shoichi <tamuki@linet.gr.jp>

Windows MIDI driver improved

- Stop sending all MIDI messages to a worker thread to be played.
  Instead, call the event handler directly (with mutual exclusion).
  This simplifies the code and reduces latency.
- Move initialization before worker thread creation - without this,
  TiMidity++ might not be initialized before the 1st message arrives
  (I actually had this problems when using VMPK, which sends a bank &
  program change message immediately on startup)

Signed-off-by: TAMUKI Shoichi <tamuki@linet.gr.jp>

Fix to fit with MinGW environment

Signed-off-by: TAMUKI Shoichi <tamuki@linet.gr.jp>

Allow setting ao backend options via environment

Let libao backend set ao options via TIMIDITY_AO_OPTIONS environment variable.
The variable accepts "key=value" pairs seperated by commans.
For example:
        TIMIDITY_AO_OPTIONS="use_mmap=1,id=1" for ALSA ao backend.
        TIMIDITY_AO_OPTIONS="server=example.com:4713"
etc.

Notes:
        1. Later options have precendance over earlier ones in libao.
        2. Spaces are not trimmed, but supplied as is.
        3. In case of "key=,", an empty string is passed to ao as value.

Signed-off-by: Yair K <cesium2@gmail.com>
Fix typo on timidity.cfg.5.

Fix typo on timidity.cfg.5. Patch sent by Kyle Guinn.

Remove tclIndex due to autogenerated file

auto_mkindex will now run with tclsh instead of wish.

Move ChangeLog to ChangeLog.3

Remove ChangeLog, it will be generated from git log.

Xaw interface: minor improvements

* Get rid of unnecessary argument to expandDir
* Free memory returned from XawListShowCurrent
* Remove outdated comment from TODO

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Include xbm bitmaps in the tcl interface itself

Include xbm bitmaps in the tcl interface itself rather than as files in the
system.

Also, include tcl.h a bit later to avoid a redefine warning on reverb.h.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Always use local OSSv3's midi sequencer definitions

From NetBSD pkgsrc: http://www.netbsd.org/docs/software/packages.html

Both the midi output mode and the server interface try to collect some
local MIDI macro definitions instead of using its own. Problem is some
systems have a 'compatibility' soundcard.h file which is detected by
configure but doesn't have the MIDI defines (e.g. NetBSD's libossaudio).
So always use the internal defines.

Add a comment to server_defs.h to show how to still get the defines from
the system if desired.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Bring in some fixes from mikmod.hg.sf.net

Bring in some fixes from mikmod.hg.sf.net for CVE-2009-0179, CVE-2009-3996,
CVE-2010-2546 and CVE-2010-2971. Also, staticize most functions as in that
version. Could perhaps use a bit more testing.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Revert for required ctl_speana_data function

From gentoo timidity++-2.14.0 packages:

This will revert part of the commit from:

http://timidity.git.sourceforge.net/git/gitweb.cgi?p=timidity/timidity;a=commit;h=e73b53437dbc1e57e61dd4d0c1407784797b08d9

Because otherwise TiMidity++ simply won't build as per:

../interface/libinterface.a(xskin_c.o): In function `ctl_event':
xskin_c.c:(.text+0x17c): undefined reference to `ctl_speana_data'
collect2: error: ld returned 1 exit status

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Sync with CVS

Bring in the ChangeLog, NEWS and configure.in updates from the 2.14.0 release.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Xaw: Fix a possible crash

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Xaw: Correctly cut off text in the trace'sinstrument name column correctly

The attached diff makes the trace cut off text in the instrument name column
correctly in the Xft case.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Xaw: Make a double-click open/save a file in the dialogs

Also remove some useless casts, and a no longer necessary action
(do-popdown()).

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Continue the dynamic interface loading loop if dl_load_file fails

Earlier, it simply returned NULL in that case, which meant that if an dlopen
failed at an dynamic interface loaded before arriving at the desired
interface (say, because of missing dependencies or bad symlink) one couldn't
load said desired (dynamically-linked) interface at all.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Various fixes

-make the play button activate when autostarting a saved list and when saving
 a file (two chunks).
- add a few consts (not needed by most compilers)
- add some keysyms of the numeric keyboard to the resource files

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

xaw interface: various fixes

-There was an off-by-one in the ARGB converter.
-Make sure the pause button can't be set when not playing
-and visa versa for the play button
-leak less memory

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

xaw: Setup _NET_WM_ICON correctly

Having upgraded to a new KDE version, I noticed the task switcher didn't
display timidity's thumbnail. Apparently it doesn't acknowledge the
WM_HINTS bitmap.

So the attached diff includes a trivial converter (1bit bitmap->ARGB) to set
up _NET_WM_ICON correctly. It uses the simple timidity.xbm icon (I also had a
version using the color icon from pixmap/timidity.xpm but I prefer this icon -
I could add that as an option).

The diff also includes some #include rationalization and resource renaming
("noPlaying" had a different spelling in .ad files vs the code).

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Sync ChangeLog with CVS

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Use AS_HELP_STRING to make the --help output more readable

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Fix type-punning in smplfile.c

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Fix the missing return value in emacs_c.c

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Add the lzma decompression support

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Add missing quotes in autoconf/utils.m4

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Add missing void argument for function declarations

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Fix the faulty check of parameters in server_c.c:control_getcmd()

The check of the array size in server_c.c:control_getcmd() is wrong
as the nparmas is incremeted after the check with MAX_GETCMD_PARAMS.
Also, there are other bugs that *nparams isn't initialized when the
first token is NULL, etc.  Overall, the code is unnecessarily tricky.

This patch simplifies the code and fixes the array size check.

Bugzila: https://bugzilla.novell.com/show_bug.cgi?id=517719
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Fix possible invalid access below the array range

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Correctly apply CurVol setting in ~/.xtimidity

Make the player correctly apply CurVol setting in ~/.xtimidity (timidity
ignores RC_CHANGE_VOLUME when not playing, so do it more directly) (all
issues except the first existed in the original 2.13.2 interface).

Also fix:
-Make the check mark disappear also the first time trace is unhidden
-Do not allow the playbutton to activate when no files are loaded

This should do for 2.14.0.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Revert old spectogram workaround

An old change of mine to miditrace.c was made to allow the spectrogram
(usually attached to 'g' key) to work in trace mode (can't remember if
the problem was in the xaw interface alone or not). Apparently, this
caused problems on the mac interface which led to the ifdef next door
(didn't notice this at the time). Now, local testing shows it's no
longer necessary on unix. So I suggest reverting it for simpler code.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Make mute show

Unmuting a channel (right click) didn't show immediatlely due to an off by one
in the snprintf in drawMute. Also another off by one in xdnd.c and some
comment and style changes.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Xaw: unbreak xdnd

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Fix compiler warnings from clang

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

configure: Fix building with X11 based userinterfaces builtin

Reported-by: Yair K. <cesium2@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Improve timidity default settings

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

xaw interface update

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

xaw-interface: Fix crash when MAX_CHANNELS > 32

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Add a trysource config file directive

Add a "trysource" config file directive, which allows to try and source
other config files, while continuing without error if the specified file
is missing.

Also cleanup the user config loading code using this new functionality.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

configure: Fix building of dynamic user interfaces

Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Update Makefile.am for autogen.sh

Add autogen.sh

Remove due to autogenerated file

Fix to fit with libpng 1.5 and newer

[PATCH 18/18] configure: Don't add -lX11 to the generic LIBS

This stops timidity itself from depending on libX11 even when build
without any interfaces builtin which need libX11.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 configure.in |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

Remove all the autogenerated files

(Revised) Disable ALSA sequencer interface polling, when it's not needed

Revert disable ALSA sequencer interface polling, when it's not needed

Revert Change sustained notes not to drop when opt_overlap_voice_allow is enabled

In the case of --enable-dynamic=emacs,alsaseq, make install timidity.el
and enable the long option of ALSA sequencer interface, respectively

Set the absolute path of timidity in timidity.el automatically

[PATCH 11/17] Make shebang paths /usr/bin/foo rather then /usr/local/bin/foo

This makes life easier for packagers, people who want to use non standard
paths will need to fix this up themselves...

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 interface/tkmidity.ptcl |    2 +-
 interface/tkpanel.tcl   |    2 +-
 script/dllutl.rb        |    2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

Support for Tcl/Tk 8.5 and autoreconf

Fix ChangeLog

[PATCH 17/17] Fix listing of dynamic interfaces in --help output

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 timidity/timidity.c |   16 ++++++++--------
 1 files changed, 8 insertions(+), 8 deletions(-)

[PATCH 16/17] Silence dlsym errors

The way the new dynamic_interface_module() function probes for dynamic
interfaces cause it to always hit a few dlsym errors as soon as you have
more then one dynamic interface, so silence these errors.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 timidity/dl_dlopen.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

[PATCH 15/17] Get rid of the dynamic_control_mode ControlMode

This is a left over from before the dynamic_interface_module() rewrite,
not only is it no longer needed its code is plain wrong, as it still
contains a prototype for the old way different dynamic_interface_module()
and tries to call it as if it were the old version, if that code path where
to ever be invoked bad things (tm) would happen. Luckily it is never used,
so simply nuke it.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 interface/Makefile.am |    6 --
 interface/dynamic_c.c |  130 -------------------------------------------------
 timidity/controls.c   |   10 ----
 timidity/timidity.c   |   31 +++---------
 4 files changed, 7 insertions(+), 170 deletions(-)
 delete mode 100644 interface/dynamic_c.c

[PATCH 14/17] Silence autofoo warnings about have_speex ac cache identifier

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 configure.in |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

[PATCH 10/17] Fix / silence various compiler warnings

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 interface/gtk_i.c    |   39 +++---
 interface/server_c.c |   12 +-
 interface/tk_c.c     |    7 +-
 interface/wrdt_tty.c |    4 +-
 interface/x_sherry.c |    4 +-
 interface/xaw_c.c    |   29 +++--
 interface/xaw_i.c    |   32 +++---
 interface/xskin_c.c  |   14 ++-
 libarc/unlzh.c       |    1 +
 libunimod/load_far.c |    4 +-
 libunimod/load_it.c  |    6 +-
 libunimod/mloader.c  |    4 +-
 libunimod/mlutil.c   |    4 +-
 timidity/ao_a.c      |    3 +-
 timidity/calcnewt.c  |   10 +-
 timidity/common.c    |   13 ++-
 timidity/effect.c    |    1 -
 timidity/esd_a.c     |    2 +-
 timidity/freq.c      |   14 +--
 timidity/instrum.c   |    3 +-
 timidity/loadtab.c   |    2 +-
 timidity/m2m.c       |   44 ++++----
 timidity/miditrace.c |    3 +-
 timidity/output.c    |    5 +-
 timidity/playmidi.c  |   19 ++--
 timidity/readmidi.c  |    5 +-
 timidity/recache.c   |    2 +-
 timidity/resample.c  |    2 +-
 timidity/reverb.c    |  336 +++++++++++++++++++++++++++++---------------------
 timidity/smplfile.c  |   10 +-
 timidity/speex_a.c   |    7 +-
 timidity/timidity.c  |   42 ++++---
 utils/net.c          |    2 +-
 utils/nkflib.c       |   18 ++-
 34 files changed, 391 insertions(+), 312 deletions(-)

[PATCH 9/9] sndfont: Work around soundfonts with missing links between stereo samples

Some sf2 files have all their link ids between stereo samples set to 0,
this patch works around that, atleast for files which have 2 matching
the matched left + right samples directly after each other in the
instruments layers list.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 timidity/sndfont.c |   57 ++++++++++++++++++++++++++++++++++++++++++++++++---
 1 files changed, 53 insertions(+), 4 deletions(-)

[PATCH 8/9] Fix a bunch of: warning: implicit declaration of function ... warnings

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 timidity/controls.c |    5 +++++
 timidity/effect.c   |    2 ++
 timidity/oss_a.c    |    1 +
 timidity/playmidi.h |    1 +
 timidity/speex_a.c  |   10 ++++++++++
 timidity/wrd.h      |    2 ++
 6 files changed, 21 insertions(+), 0 deletions(-)

[PATCH 7/9] flac_a: Fix compiling with recent flac versions

The LEGACY_FLAC test was only succeeding on windows since the
FLAC/export.h was inside a windows #ifdef block.

While at it also fix various compiler warnings, including atleast
2 real bugs:
1) The missing include for common.h meant the safe_malloc prototype was
   missing, making the compiler assume it returns an int -> fail on
   64 bits
2) The | with FLAC__STREAM_ENCODER_VERIFY_MISMATCH_IN_AUDIO_DATA in the
   error handling test was missing () and it is the wrong thing to do
   in general since this is an enum not a flags field.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 timidity/flac_a.c |   11 +++++------
 1 files changed, 5 insertions(+), 6 deletions(-)

[PATCH 6/9] Fork earlier when we're going to run deamonized in ALSA sequencer mode

If we're going to fork for daemon mode, we need to fork earlier, as
certain output libraries (pulseaudio) become unhappy if initialized
before forking and then being used from the child.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 interface/alsaseq_c.c |   17 -----------------
 timidity/timidity.c   |   23 +++++++++++++++++++++++
 2 files changed, 23 insertions(+), 17 deletions(-)

[PATCH 4/9] Various man page fixes

courtesy of Debian
---
 doc/C/timidity.1     |    8 ++++----
 doc/C/timidity.cfg.5 |   10 +++++-----
 2 files changed, 9 insertions(+), 9 deletions(-)

[PATCH 3/9] Change audio output detection order

First try the various desktop daemons we support, and only when those
are not available use the systems native sound system. Also try ALSA before
the native sound system, since the native sound system on Linux is set to
OSS, and ALSA definitely is preferable over OSS.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 timidity/output.c |   30 +++++++++++++++---------------
 1 files changed, 15 insertions(+), 15 deletions(-)

[PATCH 2/9] libao: Add a detect() function

Most Linux distros use pulseaudio by now, and libao is the best way to use
pulseaudio from timidity. This patch adds a detect function to libao, which
will make libao autodetect succeed only when pulse is available, so that
we can prefer libao in the probe order to get pulse, without also getting
libao on systems where we should be using alsa directly.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 timidity/ao_a.c |   31 ++++++++++++++++++++++++++++++-
 1 files changed, 30 insertions(+), 1 deletions(-)

[PATCH 1/9] esd_a: Don't start ESD

Don't start ESD if it is not running yet, esp. not from detect() !

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 timidity/esd_a.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

* Fix a bug created at the previous commit that prevents alternate instruments from loading occasionally.
* Fix a bug that ignores user drum source instrument when "soundfont" directive is used.
* Fix a bug that ignores non-standard drumsets when a soundfont is specified with "soundfont" directive and the instrument is loaded as needed.
* Fix potential uninitialized pointers access.
* Add access keys to some of twsyng menuitems.

Fix a bug that ignores non-standard drumsets when a soundfont is specified with "soundfont" directive.

Fix an error message for the bad floating point values.

Quick fix for Microsoft Security Advisory 2269637. (Windows executable)

Correct sf2 broken loop point to avoid infinite loop on resampling.

Fix insertion effect Lo-Fi making loud noise.
Workaround for Moog VCF making loud noise.
Remove obsolete filter frequency cap.

Remove obsolete filter frequency cap.

Workaround for Moog VCF making loud noise.

Fix insertion effect Lo-Fi making loud noise.

Fix infinite loop on reusing soundfont struct.

Fix ch_3tap_delay crash.

changed file name was wrong; playmidi.c => readmidi.c

Fix existing anti-popping minimum volume ramps for expression, volume, pans, etc.. Extend anti-popping minimum volume ramps to envelope amp changes.

Allow Device Numbers other than 0x10 for XG SYSTEM ON SYSEX events

playmidi.c: Allow Device Numbers other than 0x10 for XG SYSTEM ON SYSEX events
mix.c: Fix existing anti-popping minimum volume ramps for expression, volume, pans, etc.. Extend anti-popping minimum volume ramps to envelope amp changes.

timidity/sun_a.c (acntl): Fix for message PM_REQ_GETQSIZ, PM_REQ_GETFRAGSIZ.

Changelog was incomplete again. sorry

timidity/readmidi.c: Fixed crash when loading long music.
timidity/timidity.c: Fixed corruption of the default device encoding on some occasion.
timidity/w32_a.c: Fixed Windows audio driver 24-bit output.
interface/w32g_syn.c: Better tray icon removal.

Fixed Windows audio driver 24-bit output.

Fixed corruption of the default device encoding on some occasion.