Hi
On Fri, Oct 7, 2022 at 1:04 AM Richard Henderson <richard.henderson@linaro.org
<mailto:richard.henderson@linaro.org>> wrote:
On 10/6/22 05:51, Marc-André Lureau wrote:
> Hi Richard
>
> On Mon, Oct 3, 2022 at 11:39 AM Marc-André Lureau
<marcandre.lureau@redhat.com
<mailto:marcandre.lureau@redhat.com>
> <mailto:marcandre.lureau@redhat.com
<mailto:marcandre.lureau@redhat.com>>> wrote:
>
> Hi
>
> On Fri, Sep 30, 2022 at 6:10 PM Richard Henderson
> <richard.henderson@linaro.org <mailto:richard.henderson@linaro.org>
<mailto:richard.henderson@linaro.org
<mailto:richard.henderson@linaro.org>>> wrote:
> >
> > On 9/30/22 07:03, marcandre.lureau@redhat.com
<mailto:marcandre.lureau@redhat.com> <mailto:marcandre.lureau@redhat.com
<mailto:marcandre.lureau@redhat.com>>
> wrote:
> > > +static bool
> > > +set_thread_description(HANDLE h, const char *name)
> > > +{
> > > + HRESULT hr;
> > > + g_autofree wchar_t *namew = NULL;
> > > +
> > > + if (!load_set_thread_description()) {
> > > + return false;
> > > + }
> >
> > I don't understand why you're retaining this.
> > What is your logic?
> >
>
> Also, if we change the "static bool name_threads" to be true by
> default, then set_thread_description() might be called without
calling
> qemu_thread_naming()
>
> It really shouldn't hurt to keep it that way.
>
>
>
> Let me know if the current form is ok for you, thanks
I guess it's ok, sure.
Should I take that for an Ack? :)