Re: [yoshimi-user] suggested patch
- From: Kristian Amlie <kristian@xxxxxxxxxx>
- To: yoshimi-user@xxxxxxxxxxxxxxxxxxxxx
- Date: Sat, 04 Feb 2012 13:12:11 +0100
On 02/04/12 00:17, Brendan Jones wrote:
OK, here's a suggested patch for the window title. For example:
# window title = "Synth 1", using name tag parameter
yoshimi -N "Synth 1" &
# window title = "Yoshimi: params-x"
# this is the existing behaviour using the load parameters flag
yoshimi -l /mnt/yoshimi/params-x.xmz &
# window title = "Synth 1: params-x", using both
yoshimi -N "Synth 1" -l /mnt/yoshimi/params-x.xmz &
Let me know if this is OK.
The patch works great on my system, and the code looks alright to my
eyes. Just one question though: With the current patch, you remove the
Yoshimi name entirely. It would be better to still keep "Yoshimi" as a
prefix, and add the name tag after it, no? Otherwise you can't see which
program it is at all in the task bar.
--
Kristian
Other related posts: