On Tue, Jan 10, 2012 at 08:05:46AM +0000, Chris Morley wrote:
>    This patch allows one to select 'HAL pin type' 0 (no pin) or 1 (regular in
>    pin)
>    in the Glade editor - it defaults to 1 which is with the HAL pin.
It's better to use boolean type so glade will show it as checkbox. Along with 
more
obvious name (like without_hal_pin) it will be more easy to understand.
 
>    Ultimately I think we should allow a third option - an LED with an out pin
>    that follows the
>    LED state - but this patch does not include this option.
It's just a led with input pin. Is not it?

Otherwise patch looks ok.

                                Pavel

------------------------------------------------------------------------------
Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex
infrastructure or vast IT resources to deliver seamless, secure access to
virtual desktops. With this all-in-one solution, easily deploy virtual 
desktops for less than the cost of PCs and save 60% on VDI infrastructure 
costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox
_______________________________________________
Emc-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/emc-developers

Reply via email to