sdbus++: async: make calculator server match sync behavior

Delete toy code and make the async implementation match the sync
implementation (except some method calls are "async").

Signed-off-by: Pat

sdbus++: async: make calculator server match sync behavior

Delete toy code and make the async implementation match the sync
implementation (except some method calls are "async").

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I8cdc9fb8d064e6e10c025b2fce0deef549942941

show more ...

sdbus++: async: server: generate method-call fn

Add binding generation for method-call functions.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I34ac44ae0cdb43a18b96a4ddc5e67df4971

sdbus++: async: server: generate method-call fn

Add binding generation for method-call functions.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I34ac44ae0cdb43a18b96a4ddc5e67df4971da939

show more ...

sdbus++: async: server: make properties protected

When the implementation wants to override the property set/get
methods, they likely want to access the underlying storage. Move
them to be protecte

sdbus++: async: server: make properties protected

When the implementation wants to override the property set/get
methods, they likely want to access the underlying storage. Move
them to be protected rather than private so they can be modified.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ib7e752e1fa6e0f358f6e663aa51c2cf0c93118b7

show more ...

sdbus++: async: server: generate set-property fn

Add binding generation for set-property functions.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I3fce183f668ea339a035b8186630ffc04

sdbus++: async: server: generate set-property fn

Add binding generation for set-property functions.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I3fce183f668ea339a035b8186630ffc04aa9af82

show more ...

sdbus++: async: server: generate get-property fn

Add binding generation for get-property functions.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I60d44b25d718373acfd581faf158f45f5

sdbus++: async: server: generate get-property fn

Add binding generation for get-property functions.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I60d44b25d718373acfd581faf158f45f5b5430e9

show more ...

sdbus++: async: server: generate signal emit fns

Populate enough of the generator to define the basic class structure
and generate bindings for emitting signals.

Signed-off-by: Patrick Williams <pa

sdbus++: async: server: generate signal emit fns

Populate enough of the generator to define the basic class structure
and generate bindings for emitting signals.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: If62c323f460fc8c73a3aca495e5b89cd84bab32a

show more ...

sdbus++: async: server: add generator stub

Add the needful changes in the tools and meson-generator to
create an asynchronous server binding header.

Signed-off-by: Patrick Williams <patrick@stwcx.x

sdbus++: async: server: add generator stub

Add the needful changes in the tools and meson-generator to
create an asynchronous server binding header.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I0607aefc006eafb9224d92b887cf2d3b30c8da06

show more ...

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

Change-Id: I5daa012bf76924eb7a7d22ed31b6b77ad2f723df
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

sdbus++: async: client: fix client_t usage

When using the "Alternatively,..." syntax from calculator-client, we
end up with the compiler error:

```
example/gen/net/poettering/Calculator/client.hpp:

sdbus++: async: client: fix client_t usage

When using the "Alternatively,..." syntax from calculator-client, we
end up with the compiler error:

```
example/gen/net/poettering/Calculator/client.hpp:149:40: error: ‘constexpr sdbusplus::client::net::poettering::details::Calculator<Proxy>::Calculator(sdbusplus::async::context&, Proxy) [with Proxy = sdbusplus::async::proxy_ns::proxy<true, true, false, false>]’ is private within this context
149 | std::forward<Args>(args)...)
```

Fix this by making the generated alias class a friend of the details
class, in addition to the client_t proxy. Add a compile test to ensure
this works. Fix up a few whitespace alignment to better match
clang-format expectations.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ie0def6e6c96acc287c002c157d93016b4a79015c

show more ...

sdbus++: async: client: example: add a few comments for clarity

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Idf21e9652c005203a3f563b46605354e4711183d

sdbus++: async: client: add generated property calls

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I25ad8c0c167af0a55e235e5a7b4a46a3019d94ae

sdbus++: async: client: move context to constructor

The async::context is needed for every client invocation, since that
is how the async task is generated. Rather than pass it in as a
parameter on

sdbus++: async: client: move context to constructor

The async::context is needed for every client invocation, since that
is how the async task is generated. Rather than pass it in as a
parameter on each function invocation, add it as a constructor to the
client-proxy object.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ic1c8ec2260b7d4774649de9559df466058a50e14

show more ...

sdbus++: async: client: add generated method calls

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I47cf5bc96b545dfe3f917655b5b94db8dac9c101

sdbus++: add start of an async client

Add generator mako templates to generate a simple async client, a
corresponding implementation of the 'calculator' interface as a
client, and a client aggregati

sdbus++: add start of an async client

Add generator mako templates to generate a simple async client, a
corresponding implementation of the 'calculator' interface as a
client, and a client aggregation class (client_t) similar to the
server aggregation class we have (sdbusplus::server::object_t).

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I6795b2d051bf4d759cb319a9388b23e29d140244

show more ...

sdbus++: move interface name to common header

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I887b85d47e4e380426973f217bb94b1bec69f7aa

sdbus++: add common header file

There are some items generated under server and client which are
overlapping and some currently in server that will also be needed
for async bindings. Create a commo

sdbus++: add common header file

There are some items generated under server and client which are
overlapping and some currently in server that will also be needed
for async bindings. Create a common header file that they can
all include in order to reduce the content generated across all these
types.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Id7af8542613503dbd745158c89892ca252eadf5a

show more ...

sdbus++: change binding namespace format

Change the generated binding namespace name such that the interface
`xyz.openbmc_project.Foo` becomes
`sdbusplus::server::xyz::openbmc_project::Foo` instead

sdbus++: change binding namespace format

Change the generated binding namespace name such that the interface
`xyz.openbmc_project.Foo` becomes
`sdbusplus::server::xyz::openbmc_project::Foo` instead of
`sdbusplus::xyz::openbmc_project::server::Foo`. This both simplifies
the namespace generation and makes the correlation from interface to
binding name much more obvious.

An alias namespace to the old name is still generated so that users
have a chance to migrate.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Id82a5b6dcb26be8644b04bf14ddaf8a58dad8479

show more ...

Use more specific includes

We shouldn't be including all of asio, just the parts we need.

Change-Id: I15f5ffa40311644df98fcb9164cac529a423d232
Signed-off-by: Ed Tanous <edtanous@google.com>

Fix meson warning

Meson warns on the calls to run_command.

WARNING: You should add the boolean check kwarg to the run_command call.
It currently defaults to false, but it will default to true in fu

Fix meson warning

Meson warns on the calls to run_command.

WARNING: You should add the boolean check kwarg to the run_command call.
It currently defaults to false, but it will default to true in future
releases of meson. See also:
https://github.com/mesonbuild/meson/issues/9300

This commit does as meson recommends, and adds check: true to all
run_command calls, which will be the default.

Change-Id: I281df6ed32c935ce7d3664a0ad628cc81dbac3a8
Signed-off-by: Ed Tanous <edtanous@google.com>

show more ...

Remove static from inline variables

These do not need to be static in this context. They are used to
construct a message and that's it, constexpr is better.

At the same time, change the auto on de

Remove static from inline variables

These do not need to be static in this context. They are used to
construct a message and that's it, constexpr is better.

At the same time, change the auto on depth to int32_t, because that is
the correct API.

Change-Id: Ie2fc84eb2fee13af4c898854b822d8dbd190f5a4
Signed-off-by: Ed Tanous <edtanous@google.com>

show more ...

Remove dead code paths

There's a few places where we call return after another return
statement, or after a method that doesn't return. These are not
required, and can be removed.

Change-Id: I0917

Remove dead code paths

There's a few places where we call return after another return
statement, or after a method that doesn't return. These are not
required, and can be removed.

Change-Id: I091716bd0eb8ad6aec7a54b66d175b348cbe00e0
Signed-off-by: Ed Tanous <edtanous@google.com>

show more ...

Remove some extra semi-colons in tests

These snuck in here and cause clang warnings. Fix them.

Change-Id: I526cfee1ebc799eae48b92e790273f31462c655f
Signed-off-by: Ed Tanous <edtanous@google.com>

Remove unused variable

Clang correctly notes that ioc is private and unused. Remove it.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I1aa16b3f3c4e08d8c48205c9e060f6e1087956d4

shell: reformat with beautysh

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I2717d31983fd17fab557bb92fc55e08e248f1159

prettierrc: update config

Copy org-wide config from openbmc-build-scripts and reformat
accordingly.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ie142989f03f7d702b48f33bbf1c50b3b2

prettierrc: update config

Copy org-wide config from openbmc-build-scripts and reformat
accordingly.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ie142989f03f7d702b48f33bbf1c50b3b2f1f47c4

show more ...