メモリー管理とc++11 以上の使用

[hsdk123]

内部コードを見ていく中で気になった点がありまして、新たなIssueを作りました。

https://github.com/SpriteStudio/SpriteStudio6-SDK/blob/develop/Common/Helper/IsshTexture.h#L26

を含め、コードベースの多くのところに new, deleteとリファレンスカウンティングの痕跡が見られますが、
これらをみるとメモリーリークの可能性を感じ心配になるところがあります。

c++11以上で提供されるunique_ptr / shared_ptrの使用をお願いしたいところですが、
何かしらの理由でc++11以上のサポートが難しい場合は、少なくともc++03のshared_ptrライブラリーの使用などをお願いできればと思っております。

ゲームエンジンなどに組み込む場合は数千以上の個体を生成・破壊することも多く、結果的にメモリーリークの可能性はかなり重要な項目として保守的なスタンスを取らざるを得ないところがありますので、ご検討お願いできればうれしいです。

それから、class, structのほぼすべてが現状だとnamespaceに囲まれていないことも将来の維持補修が多少気になるので、改善できる余地があれば是非お願いしたいです。

[SpriteStudio]

@hsdk123 こちらでの回答をお待たせしてしまい大変申し訳ありませんでした。

開発チームでご要望に沿いつつレガシーなプラットフォームでも使える形への改修を検討していました。
仰る名前空間の定義と C++11以上で利用可能なスマートポインタ使用への修正、およびレガシー向けに該当箇所のシンプルさを維持できるようであれば new/delete 利用にスイッチできる形を想定しています。
着手時期は早くて８月中旬の予定です。

お待たせすることになり申し訳ありませんがどうかご了承ください。
状況にアップデートがあり次第お知らせいたします。

[hsdk123]

スイッチできる形

了解しました、確かに両方へのサポートに対応するにはそうした方が一番確実だと感じます。
いろいろと大変でしょうけど、更新を楽しみにします！
着手できる日付まで残していただきありがとうございます。

[SpriteStudio]

@hsdk123 様
状況の報告が遅くなってしまい大変申し訳ありません。
現在開発担当が別件の不具合等の対応に当たっており本件は対応できていません。
9月後半に再開予定の見込みです。
大変申し訳ございませんが何卒ご了承ください。

[hsdk123]

分かりました、日程が少しずつ遅れている様子が若干不安でもありますが、
とりあえず当初のお話し通り今年はSpriteStudioの実装を優先できるようにしておきます。
作業開始の際に作業完了までどれほどかかりそうかの予測も軽くお願いできればこちら側も動きやすくなるかと思います。

[SpriteStudio]

@hsdk123 大変おまたせしてしまい申し訳ないです。
こちらようやく着手した状況です。
完了目処については今週中にお知らせいたします。
ご連絡および進捗が遅くなり重ねてお詫び申し上げます。

[SpriteStudio]

@hsdk123 完了時期は凡そ2～3週間後(10/15~22)の見込みです。
namespace化の対応が終わったところで一度PUSHし、次にスマートポインタ対応に移ります。
完了の報告までもうしばらくお待ち下さい。 🙇

[hsdk123]

@SpriteStudio 了解しました、作業の完了後、取り組みする過程の正規化がまた完了したら、
こちら側で引き継いで導入をはじめます。できれば年末までには導入を完了させたいので、よろしくお願いします！

[SpriteStudio]

承知しました！
ありがとうございます。
こちらこそよろしくお願いいたします。 :bow

[SpriteStudio]

@hsdk123
お待たせしております。
進捗としては、namespace化とスマートポインタへの置き換えが一段落しまして動作テストに入りました。
Viewer2 のほうで正しく読み込み、再生ができることを確認する等です。
こちらも完了しましたらご報告しますので今しばらくお待ち下さい。🙇

[hsdk123]

@SpriteStudio 確認しました！　該当PRにいくつかのコメントを参考に追加しました。

[SpriteStudio]

レビューおよびご意見をいただきありがとうございます。

#82

現在の所、std::variant など C++11 を超える規格でサポートされる機能については、レガシーなプラットフォームでビルドしているSDKユーザーにも配慮する必要があるため、当面は対応できない点、申し訳ありませんがご了承ください。

また、auto や参照演算子への書き換えなどについては、修正コストおよびSDKの既存ユーザーのマージコストの観点から直近の対応が難しいため、後の課題とさせてください。

現状ご希望に添えない点、申し訳ありませんが何卒ご了承ください。