On 05/14/2013 10:54 PM, MORITA Kazutaka wrote: > + if (!oid_is_readonly(oid)) > + return -1; The extra check inside get/set isn't necessary, caller should take care of it since it is internal API. Thanks, Yuan