Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CNativeT に auto_sprintf の機能を持たせたい #160

Closed
m-tmatma opened this issue Jun 22, 2018 · 3 comments
Closed

CNativeT に auto_sprintf の機能を持たせたい #160

m-tmatma opened this issue Jun 22, 2018 · 3 comments
Labels
enhancement ■機能追加 refactoring リファクタリング 【ChangeLog除外】
Milestone

Comments

@m-tmatma
Copy link
Member

CNativeT に auto_sprintf の機能を持たせたい

現状では auto_sprintf してから CNativeT.AppendString しているが、
CNativeT に auto_sprintf 相当の機能があればバッファの管理、初期化しなくていいし
コードが見やすくなる。

#147 のような初期化漏れに伴う不具合の発生も未然に防げる

@m-tmatma m-tmatma added the refactoring リファクタリング 【ChangeLog除外】 label Jun 22, 2018
@kobake
Copy link
Member

kobake commented Jun 23, 2018

あ、それもう手元には実装あるけど取り込むタイミング図ってました

@kobake
Copy link
Member

kobake commented Jun 23, 2018

sakura-editor/management-forum#4
これの結論にもよるかな、とも思ってましたがこれの結論待たずに先に実装入れるコストはほぼ無い(もう実装しちゃってるものあるので)取り入れちゃっても良いかも、というのが今の時点の所感です。

@m-tmatma
Copy link
Member Author

m-tmatma commented Sep 2, 2018

#303 で対応済みなのでクローズ

@m-tmatma m-tmatma closed this as completed Sep 2, 2018
@ds14050 ds14050 added enhancement ■機能追加 refactoring リファクタリング 【ChangeLog除外】 labels Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ■機能追加 refactoring リファクタリング 【ChangeLog除外】
Projects
None yet
Development

No branches or pull requests

3 participants