Fail immediately enabling/disabling unregistered technology

2010-05-11 Thread Forest Bond
Previously, calling EnableTechnology or DisableTechnology with an unregistered technology type caused the call to fail silently and eventually timeout. With this patch, a "Not Registered" error is returned immediately instead. --- src/connman.h |1 + src/manager.c |6 ++ src/notifie

Re: Fail immediately enabling/disabling unregistered technology

2010-05-11 Thread Marcel Holtmann
Hi Forrest, > Previously, calling EnableTechnology or DisableTechnology with an > unregistered technology type caused the call to fail silently and > eventually timeout. With this patch, a "Not Registered" error is > returned immediately instead. good catch. > --- > src/connman.h |1 + >

Re: Fail immediately enabling/disabling unregistered technology

2010-05-11 Thread Forest Bond
Hi Marcel, On Tue, May 11, 2010 at 08:22:50PM +0200, Marcel Holtmann wrote: > Hi Forrest, One "r" is plenty. ;) > > diff --git a/src/notifier.c b/src/notifier.c > > index 283..9f4aa3e 100644 > > --- a/src/notifier.c > > +++ b/src/notifier.c > > @@ -401,10 +401,9 @@ void __connman_notifier_of

[PATCH] Fail immediately enabling/disabling unregistered technology

2010-05-11 Thread Forest Bond
Previously, calling EnableTechnology or DisableTechnology with an unregistered technology type caused the call to fail silently and eventually timeout. With this patch, a "Not Registered" error is returned immediately instead. --- src/connman.h |1 + src/manager.c |6 ++ src/notifie

Re: [PATCH] Fail immediately enabling/disabling unregistered technology

2010-05-11 Thread Marcel Holtmann
Hi Forest, > Previously, calling EnableTechnology or DisableTechnology with an > unregistered technology type caused the call to fail silently and > eventually timeout. With this patch, a "Not Registered" error is > returned immediately instead. patch has been applied. Thanks. Regards Marcel