search

Siv3D / OpenSiv3D

C++20 framework for creative coding 🎮🎨🎹 / Cross-platform support (Windows, macOS, Linux, and the Web)
https://siv3d.github.io/
MIT License
1k stars 138 forks source link

MultiPolygonにarea()とperimeter()を追加(#1185) #1187

Closed Ogame3334 closed 7 months ago

Ogame3334 commented 7 months ago

MultiPolygonにnoexceptなconstメンバ変数であるarea()とperimeter()を実装

Ogame3334 commented 7 months ago

フィードバックありがとうございます！ 他の実装(MultiPolygon& MultiPolygon::scaleAt(const Vec2 pos, const Vec2 s)など)を参考にし、そちらでは*thisを用いていたのでそちらにしたのですが、他の部分の実装とどう異なるのでしょうか？

Reputeless commented 7 months ago

ご指摘のとおりです。他の箇所も m_data を使うよう修正すべきです。 他の箇所を修正するコミットも、本件の完了後に別途、あるいは今回の pull-request に含めてもらえると大きな貢献になります。

Reputeless commented 7 months ago

Merged. Thanks!

◆ 初めて Siv3D にコミットした方へのご案内

コミッタの方の名前を AUTHORS に記載します。
表示する名前をお知らせください。特に希望が無い場合は GitHub プロフィールの ID が使われます。

◆ Organization への招待について

OpenSiv3D 本体および Siv3D ドキュメントのリポジトリにコミットをした方、その他顕著な貢献をされた方には、GitHub の Siv3D Organization メンバー への招待が送られます。
招待を受諾し、上記ページで自身のメンバー参加表示設定を「Public」に変更すると、GitHub の自身のプロフィールページに Siv3D のアイコンが表示されます。「Private」のままだと、自身と他のメンバーにしかアイコンは表示されません。

Ogame3334 commented 7 months ago

ありがとうございます！ 表示する名前は「緑獺おがめ」でお願いします！