trema / pio

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

Fix Parser#read to return ethernet_header instead of fail #246

Closed shun159 closed 9 years ago

shun159 commented 9 years ago

ref: #242

yasuhito commented 9 years ago

ありがとうございます。テストがないので、feature/open_flow_read.feature に IPv6 のデータか何かで一行追加してもらえますか?

shun159 commented 9 years ago

@yasuhito 承知いたしました。

shun159 commented 9 years ago

@yasuhito テストを追加してみました。 よろしくお願いします。

yasuhito commented 9 years ago

@shun159 すいません、テストを feature/open_flow_read.feature に足すというのはまちがいでした。書いてもらった部分をテストが通っていません。feature/parser.feature というのを作ってもらったほうが良いと思います。

shun159 commented 9 years ago

@yasuhito 承知いたしました。ありがとうございます。

shun159 commented 9 years ago

@yasuhito featureを追加してみました。いかがでしょうか??

yasuhito commented 9 years ago

@shun159 ありがとうございます。EthernetFrameがいちばんしっくり来ますね。コミットを 1 つにまとめてもらえればマージしておきます。

shun159 commented 9 years ago

@yasuhito まとめました。mergeよろしくお願いします。