search

tacigar / maidroid

Minetest modpack: Provides cute maid robots
GNU Lesser General Public License v2.1
14 stars 12 forks source link

Improve dummy_item #161

Closed chu-hai closed 7 years ago

chu-hai commented 7 years ago

BBSで色々書いていたものをコードにするとこんな感じになります。 minetest.get_objects_inside_radius()を使用しないでMaidroid側が設定したObjectRefを 直接使用します。

微妙に怖い実装ですが……。

tacigar commented 7 years ago

なるほど... こっちの方が on_step も軽いし良さそうですね 😄

微妙に怖い実装ですが……。

どこらへんが怖いでしょうか？

chu-hai commented 7 years ago

どこらへんが怖いでしょうか？

ObjectRefを保存しているMODを他に見たことが無かったのでちょっと気になった感じです。 二重のnilチェックを行っているので大丈夫だとは思いますが。

tacigar commented 7 years ago

ObjectRefを保存しているMODを他に見たことが無かったのでちょっと気になった感じです。

なるほど...確かに... でもまあ正しく動作してますし, マージしますね！