[haiku-commits] Re: BRANCH jessicah-github.synergy [da8df3e] src/add-ons/input_server/devices/synergy
- From: Jessica Hamilton <jessica.l.hamilton@xxxxxxxxx>
- To: haiku-commits@xxxxxxxxxxxxx
- Date: Wed, 1 Oct 2014 00:33:34 +1300
On 1 October 2014 00:24, Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> wrote:
> Am 30.09.2014 13:14, schrieb Jessica Hamilton:
>>
>> On 30 September 2014 20:58, Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> wrote:
>>>
>>> Am 30.09.2014 06:01, schrieb jessicah-github.synergy:
>>>>
>>>> + if (find_directory(B_USER_SETTINGS_DIRECTORY, &path) == B_OK)
>>>> + path.Append("kernel/drivers/synergy");
>>>
>>> Why kernel/drivers? It's not a kernel add-on, so its settings file should
>>> not be in that directory.
>>
>> As far as I know, this is how the driver_settings API works. And gives
>> a human readable settings file.
>
>
> Huh? I don't question the use of the driver settings API, just the location
> of the settings file.
Ohh, I see, the way the header file was written, it looked like it
didn't take a full path as an option. Grokking search for the
implementation revealed otherwise.
I shall indeed change it :-)
Other related posts:
