Re: [BlueZ PATCH v1] doc/advertisement-monitor-api: Rename methods and shorten the preset strings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Marcel and Luiz,

Per Marcel's suggestion, I renamed the functions and refined the
string values of  features and monitor types. Can you help with
reviewing this?

Thanks,
Miao

On Tue, May 5, 2020 at 5:54 PM Miao-chen Chou <mcchou@xxxxxxxxxxxx> wrote:
>
> This renames RegisterApplication/UnregisterApplication to
> RegisterMonitor/UnregisterMonitor and shortens the strings used in some
> properties.
> ---
> Hi Maintainers,
>
> This patch updates the experimental API for advertisement monitoring.
>
> Thanks,
> Miao
>
>  doc/advertisement-monitor-api.txt | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/doc/advertisement-monitor-api.txt b/doc/advertisement-monitor-api.txt
> index 604448f56..74adbfae9 100644
> --- a/doc/advertisement-monitor-api.txt
> +++ b/doc/advertisement-monitor-api.txt
> @@ -89,7 +89,7 @@ Service               org.bluez
>  Interface      org.bluez.AdvertisementMonitorManager1 [experimental]
>  Object path    /org/bluez/{hci0,hci1,...}
>
> -Methods                void RegisterApplication(object application)
> +Methods                void RegisterMonitor(object application)
>
>                         This registers a hierarchy of advertisement monitors.
>                         The application object path together with the D-Bus
> @@ -99,7 +99,7 @@ Methods               void RegisterApplication(object application)
>                         Possible errors: org.bluez.Error.InvalidArguments
>                                          org.bluez.Error.AlreadyExists
>
> -               void UnregisterApplication(object application)
> +               void UnregisterMonitor(object application)
>
>                         This unregisters advertisement monitors that have been
>                         previously registered. The object path parameter must
> @@ -118,7 +118,7 @@ Properties  array{string} SupportedMonitorTypes [read-only]
>
>                         Possible values for monitor types:
>
> -                       "patterns_with_logic_or"
> +                       "or_patterns"
>                                 Patterns with logic OR applied. With this type,
>                                 property "Patterns" must exist and has at least
>                                 one pattern.
> @@ -130,7 +130,7 @@ Properties  array{string} SupportedMonitorTypes [read-only]
>
>                         Possible values for features:
>
> -                       "controller-based-monitor-by-patterns"
> +                       "controller-patterns"
>                                 If the controller is capable of performing
>                                 advertisement monitoring by patterns, BlueZ
>                                 would offload the patterns to the controller to
> --
> 2.24.1
>



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux