[Libosinfo] [PATCH] Marginally simplify the code to create and run a GMainLoop
Debarshi Ray
rishi.is at lostca.se
Mon Oct 30 14:24:22 UTC 2017
Hey,
On Mon, Oct 16, 2017 at 02:20:59PM +0200, Christophe Fergeau wrote:
> Just took another look at it, and we don't have this situation currently
> in libosinfo codebase, so your patch should be fine.
>
> Acked-by: Christophe Fergeau <cfergeau at redhat.com>
My point was that this situation should never arise in any
asynchronous API in the first place. :)
GTask makes it pretty hard to get into this situation. Otherwise we
could have even asserted against this eventuality.
The documentation around how a GAsyncReadyCallback should be invoked
has been clarified now:
https://bugzilla.gnome.org/show_bug.cgi?id=783825
> Have you sent the follow-up that you mention to use a non-default
> GMainContext for these operations? I have memories of seeing such a
> patch, but could not find it on the mailing list.
I never sent it to the mailing list because I was going to base it on
this initial simplification. I'll do it as soon as I have some time. I
guess I should also rebase this one against current master.
By the way, I don't have commit access, so somebody has to push the
patches for me.
Cheers,
Rishi
More information about the Libosinfo
mailing list