[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2 5/5] libxl: Add explicit cast to libxl_psr_cat_set_cbm
On Tue, 2016-01-19 at 14:21 +0000, Ian Campbell wrote: > On Tue, 2016-01-19 at 14:06 +0000, Ian Jackson wrote: > > I am not a fan of the cast as a solution.ÂÂI would rather, prefer > > to > > regularise the situation.ÂÂIf my co-maintainers agree about the > > desirability of expecting libxl callers to use enum values from Xen > > public headers, > > libxl_shutdown_reason has the same issues, libxl_tsc_mode also might, > as > might libxl_timer_mode. I'm not sure if there are others. > (FTR) libxl_scheduler too, probably? > I think we > generally handle all these the way psr is handled today (with casts > and/or > explicit conversion switches). > and in fact, libxl_scheduler --touched in another patch of this series-- is also handled in a similar way, and in that case, you seemed to be fine with it? (and I'm not complaining, I'm just genuinely confused :-) ) Dario -- <<This happens because I choose it to happen!>> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) Attachment:
signature.asc _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |