On 10/7/22 02:52, Marc-André Lureau wrote:
Hi
On Fri, Oct 7, 2022 at 1:04 AM Richard Henderson <[email protected]
<mailto:[email protected]>> 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
<[email protected]
<mailto:[email protected]>
> <mailto:[email protected]
<mailto:[email protected]>>> wrote:
>
> Hi
>
> On Fri, Sep 30, 2022 at 6:10 PM Richard Henderson
> <[email protected] <mailto:[email protected]>
<mailto:[email protected]
<mailto:[email protected]>>> wrote:
> >
> > On 9/30/22 07:03, [email protected]
<mailto:[email protected]> <mailto:[email protected]
<mailto:[email protected]>>
> 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? :)
Yes, that was sloppy.
Acked-by: Richard Henderson <[email protected]>
r~
