Patch for selecting multiple devices

6 views
Skip to first unread message

tjl

unread,
Dec 24, 2011, 6:24:46 PM12/24/11
to gd...@googlegroups.com
Here' a patch for selecting one of multiple gdigi devices.

tjl
gdigi_select_device.diff

Tomasz Moń

unread,
Jan 4, 2012, 1:06:19 PM1/4/12
to gd...@googlegroups.com
On Sun, Dec 25, 2011 at 12:24 AM, tjl <tim.j....@gmail.com> wrote:
> Here' a patch for selecting one of multiple gdigi devices.

The Doxygen comment for select_device_dialog() is wrong - please fix it.
Also, please use "DigiTech device" instead of "gdigi device".

Please include your full name (so I can properly attribute your contribution).

Tim LaBerge

unread,
Jan 4, 2012, 10:57:06 PM1/4/12
to gd...@googlegroups.com
Hi Tomasz,

Here's an updated diff. For attribution, you can use: Tim LaBerge.

To test, you can force the conditional in get_digitech_devices() to be true, which will result in all sound cards being added to the list of devices.

Thanks,

Tim



--
You received this message because you are subscribed to the Google Groups "gdigi" group.
To post to this group, send email to gd...@googlegroups.com.
To unsubscribe from this group, send email to gdigi+un...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/gdigi?hl=en.


select_device.diff

Tomasz Moń

unread,
Mar 26, 2012, 10:21:54 AM3/26/12
to gd...@googlegroups.com
On Thu, Jan 5, 2012 at 4:57 AM, Tim LaBerge <tim.j....@gmail.com> wrote:
> To test, you can force the conditional in get_digitech_devices() to be true,
> which will result in all sound cards being added to the list of devices.

Finally, I have managed to test this patch. It works perfectly fine.

I have just commited it:
https://github.com/desowin/gdigi/commit/35e42dc65351ae4079338f915ce75c0baa868f80

Thanks!

Reply all
Reply to author
Forward
0 new messages