search

trema / pio

Packet parser and generator in Ruby
27 stars 11 forks source link

Add support for OFPXMT_PACKET_REGS #300

Closed shun159 closed 8 years ago

shun159 commented 8 years ago

The packet register fields OXM_OF_PKT_REGS(N) are used to store temporary values and information alongside the packet through pipeline processing. Each packet register is 64 bit wide and maskable.

shun159 commented 8 years ago

@yasuhito bundler が古いようです？？

shun159 commented 8 years ago

@yasuhito reviewをお願いします。

yasuhito commented 8 years ago

最新版の reek は、Ruby 2.0 のサポートをはずしたようです。 Gemfile の中で、ruby2.1 以上のときだけ reek を有効にしてもらえますか?

shun159 commented 8 years ago

Gemfile の中で、ruby2.1 以上のときだけ reek を有効にしてもらえますか?

やり方がわからないのでちょっとしらべてみます。

shun159 commented 8 years ago

一応とおっていますが、書き方イケてるかどうかお願いします。

yasuhito commented 8 years ago

2.1 ではちゃんと reek がインストールされていることを確認したので、たぶん合っていると思います。ほんとは ruby_version > 2.0 的な書き方ができるとキレイなんですが、無さそうですね (?)。

あとは、コミットの並べ替えと squash をしていただければ取り込みます。 2 つある "Update CHANGELOG" を1つにするとかですね。

shun159 commented 8 years ago

無さそうですね(?)。

なさそうですね…

あとは、コミットの並べ替えと squash をしていただければ取り込みます。

りょうかいです。

shun159 commented 8 years ago

squashしました。mergeおねがいします！