SolidDevice engine

[Christopher Blauvelt]

On Sun, Apr 27, 2008 at 11:54 PM, Christopher Blauvelt <cblauvelt at gmail.com>
wrote:

>
>
> 2008/4/27 Aaron J. Seigo <aseigo at kde.org>:
>
>> hi ...
>>
>> currently i'm seeing debug about "protocol not supported" from HddTemp
>> approximately once every second.
>>
>> turns out this is coming from the SolidDevice engine, which is used by the
>> device applet.
>>
>> even though temperature is never asked for or used, the HddTemp object is
>> created. worse yet HddTemp itself has an internal timer.
>>
>> of course engines should update themselves only on request, and the timing
>> should be defined by the consumer not any code in the engine itself.
>>
>> now, i'm not sure exactly what the intended use case for the temperature
>> monitor was and i don't like doing things that are too invasive without
>> asking for more info first. so perhaps someone (Petri? Blauvelt?) can shed
>> some light on this.
>>
>> --
>> Aaron J. Seigo
>> humru othro a kohnu se
>> GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43
>>
>> KDE core developer sponsored by Trolltech
>>
>> _______________________________________________
>> Panel-devel mailing list
>> Panel-devel at kde.org
>> https://mail.kde.org/mailman/listinfo/panel-devel
>>
>>
> I plan to change how the engine updates the temperature when it's asked
> for.  What's supposed to be happening is if it can't find the server after 4
> tries it stops, so that's obviously not working either.  I didn't realize it
> was causing so much havoc so I'll fix it sooner rather than later.
> Chris
>

Try Attached.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/panel-devel/attachments/20080428/982c153c/attachment.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hddtemp.diff
Type: text/x-diff
Size: 677 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080428/982c153c/attachment.bin