Skip to content

EthernetClientとWiFiClientをテンプレートで指定するようにした#8

Open
ainehanta wants to merge 1 commit into
esp8266from
esp8266-with-template
Open

EthernetClientとWiFiClientをテンプレートで指定するようにした#8
ainehanta wants to merge 1 commit into
esp8266from
esp8266-with-template

Conversation

@ainehanta
Copy link
Copy Markdown
Member

EthernetClient実装を抽出しました.

@ainehanta ainehanta self-assigned this Apr 14, 2017
@ainehanta
Copy link
Copy Markdown
Member Author

ainehanta commented Apr 18, 2017

ただこれはテンプレートでやるべきか,beginでインスタンスポインタ渡しかどちらが良いかわからない

結局Client系のprintlnとかはClientクラスではなく各EthenertClient/WiFiClient側でmixinしているので,
ポインタ渡しだと無理

明示的インスタンス宣言で,想定するクラス以外の指定でエラーになるようにする.
明示的インスタンス宣言しても未使用のコードに関しては最適化で削除されているみたいなので
コードサイズ等に影響はなさそう.

ただ,今後異なるTCPClientクラスが出てきた場合に対応しづらくなるので,明示的インスタンス宣言はやめる

@ainehanta ainehanta force-pushed the esp8266-with-template branch from c8ecd21 to c154097 Compare April 18, 2017 08:01
@ainehanta ainehanta requested a review from miettal August 23, 2017 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant