Re: [libvirt] [PATCH 2/5] First level of plumbing for virInterface*.

On Mon, May 11, 2009 at 03:29:42PM -0400, Laine Stump wrote: > Yes, tun interfaces too. Since this is binary data rather than a > null-terminated string, > we need to decide among the following three choices: > > 1) have a fixed length (how long? is 16 bytes long enough?) and > zero-fill the shorter ones. > > 2) Add a macLen arg to any API function that uses mac address (this will > need to be a return arg in some cases too) > > 3) Only provide the versions of the functions that accept/use ASCII mac > address args. IMHO, I would play safe at this point and pick 3) First it's sufficient, from the ASCII version people can usually derive the binary one if they really need it, but mostly I think people asked for those interfaces at the libvirt level because they wanted the ability to not mess with the low level stuff, so we should focuse on the high level. And if this proves unsufficient we can stil add new APIs based on the people feedback.