On Thu, May 29, 2025 at 01:53:07PM +0900, Michael Paquier wrote:
> On Thu, May 22, 2025 at 01:01:14PM +0900, Michael Paquier wrote:
>> I have added an open item about the plan ID part as it applies to v18,
>> adding the RMT in CC to get an opinion. If we cannot get a consensus
>> on all that, letting things as they are is still logically correct,
>> even with the -Wwarnings-format-signedness argument which is not
>> included by default currently.
>>
>> Has somebody an opinion to offer?
>
> It has been one week since this last update, and there has been
> nothing except the sound of cicadas. IMO, I think that we should just
> pull the switch and make both of these IDs signed on HEAD, taking case
> of the potential signedness warning issues.
>
> Now, I don't really want to take a leap of faith without the RMT being
> OK with that now that we are in beta1.
After reading through this thread and the latest patch set, I don't see any
strong reason for the RMT to object to this change for v18. IIUC some
extensions may need to adapt, but we're still a few months from 18.0, so
that seems okay. I vaguely recall that we've made other small
extension-breaking changes during the beta period for previous major
releases.
--
nathan
