Re: [pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-03-13 Thread Tanu Kaskinen
On Mon, 2017-03-13 at 20:34 +0100, Georg Chini wrote: > One more thing before you push your patch: In source_put_cb() > and sink_put_cb() you do not use the return value of > create_dbus_object_for_*(). There should be a (void) before the > calls to avoid warnings about the unused return value.

Re: [pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-03-13 Thread Georg Chini
On 13.03.2017 20:04, Tanu Kaskinen wrote: On Mon, 2017-03-13 at 18:45 +0100, Georg Chini wrote: On 13.03.2017 17:45, Tanu Kaskinen wrote: On Mon, 2017-03-13 at 07:57 +0100, Georg Chini wrote: Yes, it is no regression. But anyway, while you are improving it, I would use a string for the user

Re: [pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-03-13 Thread Tanu Kaskinen
On Mon, 2017-03-13 at 18:45 +0100, Georg Chini wrote: > On 13.03.2017 17:45, Tanu Kaskinen wrote: > > On Mon, 2017-03-13 at 07:57 +0100, Georg Chini wrote: > > > Yes, it is no regression. But anyway, while you are improving it, I > > > would use a string for the user configured sink as you say and

Re: [pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-03-13 Thread Georg Chini
On 13.03.2017 18:45, Georg Chini wrote: On 13.03.2017 17:45, Tanu Kaskinen wrote: On Mon, 2017-03-13 at 07:57 +0100, Georg Chini wrote: On 12.03.2017 23:07, Tanu Kaskinen wrote: On Sun, 2017-03-12 at 16:45 +0100, Georg Chini wrote: On 16.02.2017 11:09, Tanu Kaskinen wrote: "Refactor" is the

Re: [pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-03-13 Thread Georg Chini
On 13.03.2017 17:45, Tanu Kaskinen wrote: On Mon, 2017-03-13 at 07:57 +0100, Georg Chini wrote: On 12.03.2017 23:07, Tanu Kaskinen wrote: On Sun, 2017-03-12 at 16:45 +0100, Georg Chini wrote: On 16.02.2017 11:09, Tanu Kaskinen wrote: "Refactor" is the wrong word, you are changing the logic.

Re: [pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-03-13 Thread Tanu Kaskinen
On Mon, 2017-03-13 at 07:57 +0100, Georg Chini wrote: > On 12.03.2017 23:07, Tanu Kaskinen wrote: > > On Sun, 2017-03-12 at 16:45 +0100, Georg Chini wrote: > > > On 16.02.2017 11:09, Tanu Kaskinen wrote: > > > > > > "Refactor" is the wrong word, you are changing the logic. > > > Maybe redesign is

Re: [pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-03-13 Thread Georg Chini
On 12.03.2017 23:07, Tanu Kaskinen wrote: On Sun, 2017-03-12 at 16:45 +0100, Georg Chini wrote: On 16.02.2017 11:09, Tanu Kaskinen wrote: "Refactor" is the wrong word, you are changing the logic. Maybe redesign is a better verb. Indeed. I think I'll change it to "improve" ("redesign" feels a

Re: [pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-03-12 Thread Tanu Kaskinen
On Sun, 2017-03-12 at 16:45 +0100, Georg Chini wrote: > On 16.02.2017 11:09, Tanu Kaskinen wrote: > > "Refactor" is the wrong word, you are changing the logic. > Maybe redesign is a better verb. Indeed. I think I'll change it to "improve" ("redesign" feels a bit strong). > > Currently the

Re: [pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-03-12 Thread Georg Chini
On 16.02.2017 11:09, Tanu Kaskinen wrote: "Refactor" is the wrong word, you are changing the logic. Maybe redesign is a better verb. Currently the default sink policy is simple: either the user has configured it explicitly, in which case we always use that as the default, or we pick the sink

[pulseaudio-discuss] [PATCH v3 1/2] refactor default sink/source handling

2017-02-16 Thread Tanu Kaskinen
Currently the default sink policy is simple: either the user has configured it explicitly, in which case we always use that as the default, or we pick the sink with the highest priority. The sink priorities are currently static, so there's no need to worry about updating the default sink when sink