On Thu, 28 Jul 2016 13:51:28 +0900 IWAMOTO Toshihiro <[email protected]> wrote:
> At Wed, 27 Jul 2016 11:35:55 +0900, > Shinpei Muraoka wrote: >> >> This patch will revert the API of NXAction for the following report. >> https://bugs.launchpad.net/dragonflow/+bug/1599367 >> revert commit id: 3c3896fd77f274ce953dcb78795aa5cc4fde6c65 >> >> This patch reverts recently introduced start and end arguments. >> Add the utility method for ofs_nbits. >> NXActionRegLoad now takes ofs_nbits argument instead of ofs >> and nbits even with this revert. > > Personally, I agree with Yamamoto-san and for ofs and nbits, > but I think this patch is fine. > > Non-uniform API is not great, but it is more important to keep it > stable and avoid extra work for ryu users, IMO. Thanks, I'll push this patch. Let's be more careful about the API changes from now. ------------------------------------------------------------------------------ _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
