> > I can go either way on this. I remember meem made the comment that we > > should pass strings. I don't have a strong feeling any more for either > > way of doing things. > > > I am not sure what concern Meem has. But I think that make it as void * will > make the programming easier. For example, to set and get a VLAN ID field: > > dladm_set_conf_field(conf, FVLANID, &vid, sizeof (uint16_t); > dladm_get_conf_field(conf, FVLANID, &vid, sizeof (uint16_t))
As I recall, my thought was that we would store the data in ASCII anyway, so having the API allow arbitrarily-typed data seemed messy. Further, in the above calsl, I'm not sure how the routines would know to treat vid as a 16-bit unsigned integer unless it knows the type of FVLANID. But then why pass the size? -- meem
