On 3/22/19 11:11 AM, Daniel P. Berrangé wrote:
On Fri, Mar 22, 2019 at 11:03:59AM -0400, Laine Stump wrote:
On 3/19/19 8:46 AM, Daniel P. Berrangé wrote:
The virNetDevBandwidthParse method uses the interface type to decide
whether to allow use of the "floor" parameter. Using the interface
type is not convenient as callers may not have that available, but
still wish to allow use of "floor". Switch to an explicit boolean
to control its usage.

Signed-off-by: Daniel P. Berrangé <berra...@redhat.com>

Or we could just always look for floor and parse it, and decide whether or
not it's valid somewhere later in validation. But this definitely doesn't
harm anything, so
Bear in mind that this series is already huge and complicated. Alot of
things in this series had many approaches, and I tended to take the one
that was least disruptive to minimize complexity & risk of this series.


Yep. I know exactly where you're coming from :-)


--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to