Hi Miao, On Fri, Sep 11, 2020 at 11:53 PM Miao-chen Chou <mcchou@xxxxxxxxxxxx> wrote: > > Given that adapter.h refers symbols from lib/bluetooth.h and lib/sdp.h, > these two headers should be included directly in adapter.h. > > Reviewed-by: Manish Mandlik <mmandlik@xxxxxxxxxxxx> > --- > > (no changes since v1) > > src/adapter.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/adapter.h b/src/adapter.h > index f8ac20261..c70a7b0da 100644 > --- a/src/adapter.h > +++ b/src/adapter.h > @@ -26,6 +26,9 @@ > #include <dbus/dbus.h> > #include <glib.h> > > +#include <lib/bluetooth.h> > +#include <lib/sdp.h> > + > #define MAX_NAME_LENGTH 248 > > /* Invalid SSP passkey value used to indicate negative replies */ > -- > 2.26.2 Applied, thanks. Note that I did introduce a path to fix the build problem with g_dbus_proxy_get_path. -- Luiz Augusto von Dentz