◀ANSI版開発トップへ
  • 3788 ちょっとしたバグ修正パッチ
    • 3789 Re: ちょっとしたバグ修正パッチ
      • 3790 Re2: ちょっとしたバグ修正パッチ
        • 3793 Re3: ちょっとしたバグ修正パッチ

  • [3788] ちょっとしたバグ修正パッチ もか 2004年11月12日 18:42

    以下、3点の改善・修正パッチです。
    >>3787 タグジャンプのバグ
    >>data:4080 アウトラインのウィンドウ枠がもったいない
    左右のマージンを消しました。
    >>data:4131 行頭禁則に句読点を重複して多く登録すると落ちる
    すすさん提案の、重複文字を設定文字列から消してくれる便利な機能は付けていません。
    そっちのほうがいいなあ。

    ヤフーブリーフケースの fix041002b.zip です。
    とりあえずですが、あげておきます。

    • [3789] Re: ちょっとしたバグ修正パッチ げんた 2004年11月13日 03:49

      etc_uty.cppのIsFilePath()で
      > // 2004.11.10 Moca 先頭に上の文字がある場合の考慮を追加
      > if( i >= 0 ){
      > goto can_not_tagjump;
      > }else
      は変ですよね？ i>=0は常に成立しますので．
      (i == 0 || ( i > 0 && _IS_SJIS1(..))を条件に加えればif文を分離する必要は無いのでは？

      関係ないですが，jがなんだか一瞬わからなかったので他の名前に変えようっと．

      >重複文字を設定文字列から消してくれる便利な機能は付けていません。
      内部のリストに登録するときは重複チェックをするけど，設定上は重複したまま残るってことですよね．

      ---
      あと細かいことですがSetLayoutInfo()の禁則処理部にunsigned char *rというのを追加していますが，if文の中でしか使われないのでそこで定義した方がbetterと思います．

      • [3790] Re2: ちょっとしたバグ修正パッチ もか 2004年11月13日 14:12

        >は変ですよね？ i>=0は常に成立しますので．
        >(i == 0 || ( i > 0 && _IS_SJIS1(..))を条件に加えればif文を分離する必要は無いのでは？
        変ですね。i<=0と書いたつもりでした。i<0もありえませんが。
        _IS_SJIS1(..)の前に !が抜けてる以外はそれでいいですね。
        　他、ご指摘どおりなので、修正してください。m(__)m

        #ながら作業は良くないぞ自分。頭の中でC89とC++の切り替えも中途半端..

        • [3793] Re3: ちょっとしたバグ修正パッチ もか 2004年11月15日 21:52

          もう少しちゃんと見てみると、
          メ*ル
          で、メの後続バイトが0x81で、_IS_SJIS1に一致するから
          「メイル」「メール」なんていうファイルがあっても開いてしまいますね。
          そんなファイル名付けないので、再現性が相当低いですけど(^ ^;