Menu

#816 メッセージボックスのコピーがおかしい

None
closed
nobody
None
5
2015-07-14
2014-05-11
novice123
No

バッファが上書きされるのを修正しました。
http://sakura-editor.sourceforge.net/cgi-bin/cyclamen/cyclamen.cgi?log=unicode&tree=r2165

メッセージボックス表示中に参照ポインタの内容が変わっても問題ないように
ローカルにコピーするようにしました。

1 Attachments

Discussion

  • novice123

    novice123 - 2014-05-11

    変数名が間違っていたので更新しました。

     
  • novice123

    novice123 - 2014-05-11

    MessageBox()->MYMESSAGEBOX()に置換しました。

     
  • novice123

    novice123 - 2014-05-12

    MessageBox()->MYMESSAGEBOX()に置換するのはやめて、
    呼び出し側でローカルバッファにコピーするようにしました。

     
  • novice123

    novice123 - 2014-05-13

    リソース名を修正しました。

     
  • novice123

    novice123 - 2014-05-14

    抜けがあったので修正しました。

    patchを否定することになりますが、長い目で見ると
    ・LoadString()のように呼び出し側でバッファを確保
    ・戻り値をstd::tstring型
    にしたほうが安心なのではないかと。

     
  • syat

    syat - 2015-07-09

    unicode:2181 に掲載されているLR4さんのパッチを添付します。
    動作を確認したところ問題なさそうで、この問題の対策として手っ取り早いのでこちらを採用でよいんじゃないかと思います。
    メッセージボックスの多言語対応もふくみます。

     

    Last edit: syat 2015-07-09
    • novice123

      novice123 - 2015-07-13

      自分が登録したパッチは破棄してください。
      無駄なコピーが発生する可能性がありますが、
      個別でバッファを確保するよりも、
      Wrap関数でバッファを確保するほうが抜けがないと思います。

       
  • syat

    syat - 2015-07-14
    • status: open --> closed
    • Group: -->
     
  • syat

    syat - 2015-07-14

    リビジョン4022でコミットしました。

     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.