On Friday, November 8, 2013, Lennart Poettering  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, 08.11.13 10:34, Marc-Antoine Perennou (Marc-Antoine@Perennou.com) wrote:<br>

<br>
Heya!<br>
<br>
I already rebased the previous version you posted, and then applied with<br>
some changes. Is this reposting of yours more than just a rebase?<br>
<br>
I merged both commits into one, and made a couple of changes:<br>
<br>
I didn't like the log_method_call_failed() thing, because I'd much<br>
prefer having good explanatory error messages which usually require<br>
format strings and stuff. I did like log_message_creation_failed()<br>
though and moved it to bus-util.c as bus_log_create_error().<br>
<br>
sd-bus.h is a public API, where we try to avoid everything that is not<br>
C89. "bool" is C99. The message parser/append calls hence assume that<br>
booleans are "int"s, as in traditional C, in glib or even the old<br>
libdbus1. However, "bool" and "int" have different sizes, which means<br>
taking a pointer of a bool and passing that to sd_bus_message_read() or<br>
so for parsing a bool will explode. C can be ugly like hell<br>
sometimes. Anyway, fixed those.<br>
<br>
The bus transport generalization doesn't do direct PID1 connections<br>
anymore, I will add this in a alter commit.<br>
<br>
Thanks a lot for the patch!<br>
<br>
We are now down to only PID 1 left in libdus-1 land!<br>
<br>
Lennart<br>
<br>
--<br>
Lennart Poettering, Red Hat<br>
</blockquote><div><br></div><div>Was mostly a rebase + trivial cleanups, nothing important.</div><div>Thanks for the hints and fixes!</div><div><br></div><div>Marc-Antoine<span></span> </div>