Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-05-08 Thread Akihiko Odaki
On 2025/05/07 0:37, Markus Armbruster wrote: Akihiko Odaki writes: On 2025/02/06 18:48, Markus Armbruster wrote: Akihiko Odaki writes: [...] I understand we have something like this: * true: on if possible, else off * false: off (always possible) Which one is the default? It depends

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-05-06 Thread BALATON Zoltan
On Tue, 6 May 2025, Markus Armbruster wrote: Akihiko Odaki writes: On 2025/02/06 18:48, Markus Armbruster wrote: Akihiko Odaki writes: [...] I understand we have something like this: * true: on if possible, else off * false: off (always possible) Which one is the default? It depends

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-05-06 Thread Markus Armbruster
Akihiko Odaki writes: > On 2025/02/06 18:48, Markus Armbruster wrote: >> Akihiko Odaki writes: [...] >> I understand we have something like this: >> >> * true: on if possible, else off >> >> * false: off (always possible) >> >> Which one is the default? > > It depends. Some properties have

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-05-04 Thread Akihiko Odaki
On 2025/02/07 21:46, Daniel P. Berrangé wrote: On Fri, Feb 07, 2025 at 01:31:47PM +0100, Markus Armbruster wrote: Akihiko Odaki writes: [...] Let me go back to the discussion of the bool/OnOffAuto problem below: The values the command line syntax accepts are on/yes/true/y and off/no/false/n

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-07 Thread Daniel P . Berrangé
On Fri, Feb 07, 2025 at 01:31:47PM +0100, Markus Armbruster wrote: > Akihiko Odaki writes: > > [...] > > > Let me go back to the discussion of the bool/OnOffAuto problem below: > > > > The values the command line syntax accepts are on/yes/true/y and > > off/no/false/n. > > > > For the command l

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-07 Thread Markus Armbruster
Akihiko Odaki writes: [...] > Let me go back to the discussion of the bool/OnOffAuto problem below: > > The values the command line syntax accepts are on/yes/true/y and > off/no/false/n. > > For the command line syntax, you can always use on/off whether the type is > bool or OnOffAuto. In my o

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-07 Thread Markus Armbruster
BALATON Zoltan writes: > On Thu, 6 Feb 2025, Akihiko Odaki wrote: >> On 2025/02/06 18:48, Markus Armbruster wrote: This problem can be solved using an existing mechanism, OnOffAuto, which differentiates the "auto" state and explicit the

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-06 Thread Akihiko Odaki
On 2025/02/06 22:23, BALATON Zoltan wrote: On Thu, 6 Feb 2025, Akihiko Odaki wrote: On 2025/02/06 18:48, Markus Armbruster wrote: This problem can be solved using an existing mechanism, OnOffAuto, which differentiates the "auto" state and explicit th

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-06 Thread BALATON Zoltan
On Thu, 6 Feb 2025, Akihiko Odaki wrote: On 2025/02/06 18:48, Markus Armbruster wrote: This problem can be solved using an existing mechanism, OnOffAuto, which differentiates the "auto" state and explicit the "on" state. I guess you're proposing som

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-06 Thread Akihiko Odaki
On 2025/02/06 18:48, Markus Armbruster wrote: Akihiko Odaki writes: On 2025/02/06 0:29, Markus Armbruster wrote: Akihiko Odaki writes: Accept bool literals for OnOffAuto properties for consistency with bool properties. This enables users to set the "on" or "off" value in a uniform syntax w

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-06 Thread Markus Armbruster
Akihiko Odaki writes: > On 2025/02/06 0:29, Markus Armbruster wrote: >> Akihiko Odaki writes: >> >>> Accept bool literals for OnOffAuto properties for consistency with bool >>> properties. This enables users to set the "on" or "off" value in a >>> uniform syntax without knowing whether the "aut

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-06 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Wed, Jan 08, 2025 at 03:17:51PM +0900, Akihiko Odaki wrote: >> Accept bool literals for OnOffAuto properties for consistency with bool >> properties. This enables users to set the "on" or "off" value in a >> uniform syntax without knowing whether the "auto" value i

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-05 Thread Akihiko Odaki
On 2025/02/06 0:29, Markus Armbruster wrote: Akihiko Odaki writes: Accept bool literals for OnOffAuto properties for consistency with bool properties. This enables users to set the "on" or "off" value in a uniform syntax without knowing whether the "auto" value is accepted. This behavior is es

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-02-05 Thread Markus Armbruster
Akihiko Odaki writes: > Accept bool literals for OnOffAuto properties for consistency with bool > properties. This enables users to set the "on" or "off" value in a > uniform syntax without knowing whether the "auto" value is accepted. > This behavior is especially useful when converting an exist

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-01-10 Thread Akihiko Odaki
On 2025/01/10 21:16, Daniel P. Berrangé wrote: On Fri, Jan 10, 2025 at 08:31:57PM +0900, Akihiko Odaki wrote: On 2025/01/10 20:09, Daniel P. Berrangé wrote: On Wed, Jan 08, 2025 at 03:17:51PM +0900, Akihiko Odaki wrote: Accept bool literals for OnOffAuto properties for consistency with bool pr

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-01-10 Thread Daniel P . Berrangé
On Fri, Jan 10, 2025 at 08:31:57PM +0900, Akihiko Odaki wrote: > On 2025/01/10 20:09, Daniel P. Berrangé wrote: > > On Wed, Jan 08, 2025 at 03:17:51PM +0900, Akihiko Odaki wrote: > > > Accept bool literals for OnOffAuto properties for consistency with bool > > > properties. This enables users to se

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-01-10 Thread Akihiko Odaki
On 2025/01/10 20:09, Daniel P. Berrangé wrote: On Wed, Jan 08, 2025 at 03:17:51PM +0900, Akihiko Odaki wrote: Accept bool literals for OnOffAuto properties for consistency with bool properties. This enables users to set the "on" or "off" value in a uniform syntax without knowing whether the "aut

Re: [PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-01-10 Thread Daniel P . Berrangé
On Wed, Jan 08, 2025 at 03:17:51PM +0900, Akihiko Odaki wrote: > Accept bool literals for OnOffAuto properties for consistency with bool > properties. This enables users to set the "on" or "off" value in a > uniform syntax without knowing whether the "auto" value is accepted. > This behavior is esp

[PATCH v4 2/4] qdev-properties: Accept bool for OnOffAuto

2025-01-07 Thread Akihiko Odaki
Accept bool literals for OnOffAuto properties for consistency with bool properties. This enables users to set the "on" or "off" value in a uniform syntax without knowing whether the "auto" value is accepted. This behavior is especially useful when converting an existing bool property to OnOffAuto o