[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Confusing definitions and declarations of mig_dealloc_reply_port()
From: |
Svante Signell |
Subject: |
Re: Confusing definitions and declarations of mig_dealloc_reply_port() |
Date: |
Thu, 05 Nov 2015 11:13:57 +0100 |
On Thu, 2015-11-05 at 10:43 +0100, Pino Toscano wrote:
> On Thursday 05 November 2015 10:30:27 Svante Signell wrote:
> >
> > + mach_port_t port = __hurd_local_reply_port;
> > + if (port == MACH_PORT_NULL ||
> > + (&__hurd_local_reply_port != &__hurd_reply_port0 &&
> > + port == __hurd_reply_port0))
> > + port = __mach_reply_port ();
> >
> > - return __hurd_local_reply_port;
> > + return port;
> > }
>
> The current version may change __hurd_local_reply_port, while your
> proposed version does not.
The allocation could be fixed by?
port = __hurd_local_reply_port = __mach_reply_port ();
But, doing that would not improve readability much. Proposal withdrawn.
> >
> > Samuel: Regarding your previous comment:
> > > > Additionally, any strong reason to not change mig??
> > >
> > > Having to deal with the introduced incompatibility.
> >
> > Incompatibility with what, older versions of glibc/hurd/mach/mig,
> > which
> > ones?
>
> If your suggestion is to remove the argument from
> __mig_dealloc_reply_port, that isn't an option, as it would be an ABI
> break in libc.
Again, how many Hurd users would suffer from an ABI break? I think this
could be open for discussion, maybe IRC would be better than here.
- Confusing definitions and declarations of mig_dealloc_reply_port(), Svante Signell, 2015/11/03
- Message not available
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Svante Signell, 2015/11/04
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Diego Nieto Cid, 2015/11/04
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Samuel Thibault, 2015/11/04
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Svante Signell, 2015/11/04
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Samuel Thibault, 2015/11/04
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Diego Nieto Cid, 2015/11/04
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Svante Signell, 2015/11/05
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Pino Toscano, 2015/11/05
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(),
Svante Signell <=
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Samuel Thibault, 2015/11/06
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Svante Signell, 2015/11/06
- Re: Confusing definitions and declarations of mig_dealloc_reply_port(), Samuel Thibault, 2015/11/06