search

bdunderscore / ndmf

MIT License
63 stars 20 forks source link

ContextHolderを型だけpublicにしてほしい #213

Closed anatawa12 closed 5 months ago

anatawa12 commented 5 months ago

AAOでは過最適化や逆に最適化しないことを防ぐためにアバターについているすべてのコンポーネントについて登録を行っています。 ContextHolderも処理中に存在するためContextHolderも登録する必要があるのですが、public classになっていないためReflectionを使用する必要があります。 中身はprivate / internalのままで問題ないのでContextHolderをpublic classにしてほしいです

bdunderscore commented 5 months ago

Public classにしてしまうと今後削除できなくなるし、AddComponentまだきてしまうので避けたいところです。

Abstract baseや interfaceの作成を検討します