[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-dbus
Subject:    Re: Error naming convention on dbus
From:       Thiago Macieira <thiago () kde ! org>
Date:       2021-02-07 18:33:17
Message-ID: 2592003.obP2CiTvP2 () tjmaciei-mobl1
[Download RAW message or body]

On Saturday, 6 February 2021 14:40:57 PST Nyirő Gergő wrote:
> I would like to respond with an error if
> the validation fails. Which would be a better error name:
> a) `org.freedesktop.DBus.Properties.Error.ValidationError`
> b) `org.configd.Error.ValidationError`
> c) replace the standard `org.freedesktop.DBus.Properties.Set` method with a
> custom interface + method for validation

I would say (b). It's certainily not (a) because you're not the authority for 
org.freedesktop.DBus.

Whether you need a separate method or not depends on whether existing callers 
of the current interface can reliably call the enhanced one and expect it to 
work. In this particular case, I think you're safe in just adding more 
possible errors and therefore don t need a new method.

-- 
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
   Software Architect - Intel DPG Cloud Engineering



_______________________________________________
dbus mailing list
dbus@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dbus

[prev in list] [next in list] [prev in thread] [next in thread]