◀ANSI版開発トップへ
  • 608 RC4.3登録
    • 609 無意味な変更はやめて(;_;)(-_-#)
      • 610 私の考える無意味な変更
        • 611 意味はあると思うからやった
          • 612 Re: 意味はあると思うからやった
            • 614 Re2: 意味はあると思うからやった
              • 617 Re3: 意味はあると思うからやった
                • 618 Re4: 意味はあると思うからやった
                  • 619 Re5: バージョン管理の意義
                    • 620 Re6: バージョン管理の意義
                      • 626 Re7: バージョン管理の意義
                • 623 Re4: 意味はあると思うからやった
                  • 624 Re5: 意味はあると思うからやった
                    • 625 プログラム規模
    • 613 Re: RC4.3登録
      • 615 Re2: RC4.3登録
        • 616 Re3: RC4.3登録
  • [608] RC4.3登録 じぇぷろ 2001年07月13日 19:58

    ソースの不要なタブやスペースを省いたり、全角スペースをなくしたり、ファイル最後尾に /*[EOF]*/を追加したりしたらほぼ全ファイル修正になってしまったので、丸ごとアップしました。これで当分お休み。
    • [609] 無意味な変更はやめて(;_;)(-_-#) げんた 2001年07月16日 13:35

      >ソースの不要なタブやスペースを省いたり、全角スペースをなくしたり、ファイル最後尾に /*[EOF]*/を追加したり
      括弧の間にスペースを入れたり,コメントの位置をTABで揃えたり,コメントの句読点を揃えたりされると何が変わったのか全然わからなくなってしまう.差分ファイルもサイズが膨らんで全然役に立たない.
      • [610] 私の考える無意味な変更 げんた 2001年07月16日 14:28

        実行ファイルに何ら影響を及ぼさず,かつ,ソース閲覧上有意な情報価値を付加しない変更は,いたずらに差分ファイルを増加させて意味のある変更点の理解に支障を来します.

        ▼意味があると思う
        条件文追加に伴うインデントの調整
        表示文字列の修正,位置調整.
        意味のある変更に伴うその周辺の視認性向上のための変更

        ▼意味がないので避けて欲しい
        空行の空白の削除 (構造化エディタの使用を考えたら有意?)
        コメント中の全角スペース←→半角スペース変換
        TAB←→SPACE変換
        コメントの桁揃え
        コメントの表現変更
        考慮の足りない文字列一括置換
        など.

        どうしても意味のない変更をしないと気が済まないのであれば,ソースを変換ツールにかけて(ソースであるがソースでは無くなったものを)閲覧してください.変換してもスペースは見えないだろうけど(笑).
        • [611] 意味はあると思うからやった じぇぷろ 2001年07月16日 20:03

          ▼ げんたさん
          > 差分ファイルを増加させて
          差分を使わなければばいい。昔は「全くチェックしてません」とか「そのままアップしてください」ということがよくあったけどね。

          > ▼意味がないので避けて欲しい
          GPL化も近いのでここらで一度整理・統一しておくと後々までいいと思ったんですがね.(丁度皆さん新規アップの意欲も失せたようだし)
          ホワイトスペースは無視できるツールをおもちでなかったですか?どうせファイルのバージョンはあってないようなものなのだから、そのまま置き換えてくれればいいのに。信用できないなら仕方ない。
          • [612] Re: 意味はあると思うからやった げんた 2001年07月16日 20:34

            >差分を使わなければばいい。
            そう言ってしまえばその通りなんですが(汗).もう1回くらいライセンスを決定した後で全ソースコードのヘッダ書き換えになりそうな気はしますが.

            > ここらで一度整理・統一しておく
            これは否定しません.しかし,空行のスペースはオートインデントだとどうしても入ってしまうことがありますし,句読点やスペースの種別は人によってIMEの設定が異なりますので,再び同じことが繰り返されるように思います.

            ソースツリーに登録した後でこれをやられると変更点が認識しづらくなったり,差分のつもりが元ファイルより大きくなったりしますので今後は避けたいなと.毎回じぇぷろさんがチェック,修正した後で公開するなら安心?
            • [614] Re2: 意味はあると思うからやった じぇぷろ 2001年07月16日 22:22

              ¥> >差分を使わなければばいい。
              基本的に私は差分やパッチは信用してないんです。信用できない事象に実際あっているから。それに差分をあてられる人はpatch を持っている人に限られるし、それくらいなら変更ファイルのままのほうがいい。変更点が気になる人は自分でdiffしてみればいい。
              数十~数百KBは今のネットワーク環境ではあまり気にしなくてもいいと思うし。スピードは別としても料金的にはどうせ定額とかなんだろうから。
              差分をDLする人はいるのかなあ。私はパッチをあてる手間がいらないから毎回ソース一式をDLしてます。昔はそれしかなかったし。今自分が修正している個所があったとして、それにパッチをあてたとき、自分の修正箇所が消えることがあるんです。それが嫌だから、いつの頃からか、いじる時は毎回最新ソースを「まっさら」の状態からと決めてます。

              > > ここらで一度整理・統一しておく
              ソースの書き方は人によって違うのでしょうが,少なくともこの種のいろんな人が手を入れるソースの場合、プログラム内容で独自色を出すのは構わないけれども、スタイルで独自色を出すのはどんなものでしょう。元のソースのスタイルを真似て書くのが一般的だと思うのですが。そういう意味で全面的に一度それなりに統一しておけば、例えばファイルの最後には[/*EOF*/]と明示的に書いておくんだな、とわかると思うんです。たまたま見たファイルにそれが入ってないと、書かないファイルを真似るものが出てくる(増殖する)。
              統一は見やすさに繋がり、修正しやすさに繋がるというのが私の考え。一度抜本的にやっておけば、後はそれを真似るも真似ないもその人次第。何度もやる作業ではありません。抜本的な修正は何度もやるものではないです。(多分二度とないでしょう。)

              > ソースツリーに登録した後でこれをやられると変更点が認識しづらくなったり,差分のつもりが元ファイルより大きくなったり
              ホワイトスペースを無視できない?(前は無視してたハズですが。)
              • [617] Re3: 意味はあると思うからやった げんた 2001年07月16日 23:40

                >数十~数百KBは今のネットワーク環境ではあまり気にしなくてもいいと思うし。スピードは別としても料金的にはどうせ定額とかなんだろうから。
                これは必ずしもそうとは言えないと思いますけど.私なら56Kのダイヤルアップだったら2~300KBが限界ですね.それ以上のファイルはサイズだけ見てあきらめ.この辺はこれからに期待でしょう.

                > それにパッチをあてたとき、自分の修正箇所が消えることがあるんです。
                これは注意する必要があるかもしれません.基本的には衝突したら別ファイルに記録が残るんですけどね.

                >元のソースのスタイルを真似て書くのが一般的
                正直そこまで考えたことありませんでした.一部修正の時は前後に合わせますが,新規関数,新規ファイルのときは自分流で作っています.大まかなコメントの付け方とかは真似るでしょうけど,括弧の細かい位置とかスペースとタブの使い分けとか空行とかをわざわざ真似しようとは思いません.1つの関数の中で十分に見やすければそれでいいと思いますけど.

                あと,ファイル末尾の/*[EOF]*/は何の意味があるんでしょう?

                徹底しようとすればコーディング規約を定めて,しかも修正を反映するときに誰かがスタイルチェックを行わないと難しいですが,そこまで労力をかける価値は無いのではと思います.

                抜本的な変更といえば,関数・クラスのコメント整備というのも全部やった方がいいのかもしれないが,量的に厳しい.

                >ホワイトスペースを無視できない?(前は無視してたハズですが。)
                あれは私がファイルを比較して,変更点が見つからなかったので更新するのをやめたのであって,管理ツール自体はスペースを無視しません.
                • [618] Re4: 意味はあると思うからやった じぇぷろ 2001年07月17日 03:48

                  ▼ げんたさん
                  >私なら56Kのダイヤルアップだったら2~300KBが限界ですね.それ以上のファイルはサイズだけ見てあきらめ.
                  諦めが早いですね。800KBでも5、6分から10分程度だと思いますが。私は必要だと思えばDLします。ULも。困ると思えば投稿もするでしょう。

                  > 衝突したら別ファイルに記録が残るんですけどね.
                  一部が変わって一部が変わってない、斑状態ではかえって自分の修正がわからなくなるんです。

                  > >元のソースのスタイルを真似て書くのが一般的
                  > 正直そこまで考えたことありませんでした.一部修正の時は前後に合わせますが,新規関数,新規ファイルのときは自分流で作っています.大まかなコメントの付け方とかは真似るでしょうけど,括弧の細かい位置とかスペースとタブの使い分けとか空行とかをわざわざ真似しようとは思いません.
                  そういう人もいるでしょうが、統一的なほうがそろってないよりいいだろうし、揃えることが苦でない人はその方がいいと思います。苦な人までとはいいません。

                  > あと,ファイル末尾の/*[EOF]*/は何の意味があるんでしょう?
                  これも同上。はじめにそう書いてあるファイルが多かったのが最大の理由。消すのと追加するのとで手間の少ない方。意味そのものは。。。ファイル後半を修正したとき、例えば移動とかしたときうっかり他へペーストし忘れたとかのミスがなくなる(かも)。

                  > 徹底しようとすればコーディング規約を定めて,しかも修正を反映するときに誰かがスタイルチェックを行わないと難しいですが,そこまで労力をかける価値は無いのではと思います.
                  だから常のことではなく、GPLソース公開直前のお化粧としてということ。お化粧で中身が変わるわけではないが、お披露目ではお化粧することにそれなりの意味もあるのでは?(男までするかあ?ということはある)

                  > 抜本的な変更といえば,関数・クラスのコメント整備というのも全部やった方がいい
                  目の付くたびに少しずつ修正するか、あるとき気合を入れて一気にやるか。まあ、前者でしょう。私はGrepしまくってやったけど。
                  • [619] Re5: バージョン管理の意義 KENCH 2001年07月17日 09:37

                    ▼ じぇぷろさん
                    > ▼ げんたさん
                     バージョン管理の話題を私もちらほらと見かけますが、幾人もの人がばらばらにソースを修正する状況においてこそバージョン管理ソフトは適用すべきといわれます。
                     が、実際にはバージョン管理をみんなが意識して使えないとそれこそはちゃめちゃになって意味の無い状況になりがち。
                     実際の仕事でも、バージョン管理ソフトをしっかり使えるプロジェクトはバージョン管理ソフトがなくてもソースをしっかり管理できるプロジェクトだったりします(^_^;
                     単に差を取るだけであればたしかにdiffで十分かもしれませんが、いつどういう意図があって修正がかかったのか、このプロジェクトに三角する人数がもっと増えてきたときにはCVS等のツールが利いて来るのかもしれません。
                     真のオープンソース版(GPL版)公開以降はそれなりに管理するとかしてみてはいかがでしょう。
                    • [620] Re6: バージョン管理の意義 げんた 2001年07月17日 12:16

                      >実際にはバージョン管理をみんなが意識して使えないとそれこそはちゃめちゃになって意味の無い状況になりがち。
                      φ(.. )メモメモ

                      >いつどういう意図があって修正がかかったのか
                      いつどのバグを誰がどうやって直したか,どのバージョンにはどの機能が入っていてどのバグが直っていないのかがわかるといいんですが,安くてうまくて早い方法あります?

                      >真のオープンソース版公開以降はそれなりに管理
                      sourceforgeのCVSサーバが未使用なので,そちらに登録するつもりです.既に手元にあるCVSリポジトリをそのまま突っ込むつもりはありません(ライセンスが不明確な部分を含んでいるので).1.3.0をスタート地点にする予定です.
                      • [626] Re7: バージョン管理の意義 KENCH 2001年07月18日 16:51

                        > いつどのバグを誰がどうやって直したか,どのバージョンにはどの機能が入っていてどのバグが直っていないのかがわかるといいんですが,安くてうまくて早い方法あります?
                         あったら僕が聞きたい(^_^;
                         結局運用ルールを決めて、ちゃんと運用しないとだめかも。
                         でも、ちゃんと運用できないからこそCSV使うと言う人もいますけどね、、、
                • [623] Re4: 意味はあると思うからやった nakatani 2001年07月17日 21:02

                  ファイル末尾の/*[EOF]*/は
                  ▼ げんたさん
                  > >数十~数KBは今のネットワーク環境ではあまり気にしなくてもいいと思うし。スピードは別としても料金的にはどうせ定額とかなんだろうから。
                  > これは必ずしもそうとは言えないと思いますけど.私なら56Kのダイヤルアップだったら2~300KBが限界ですね.それ以上のファイルはサイズだけ見てあきらめ.この辺はこれからに期待でしょう.
                  >
                  > > それにパッチをあてたとき、自分の修正箇所が消えることがあるんです。
                  > これは注意する必要があるかもしれません.基本的には衝突したら別ファイルに記録が残るんですけどね.
                  >
                  > >元のソースのスタイルを真似て書くのが一般的
                  > 正直そこまで考えたことありませんでした.一部修正の時は前後に合わせますが,新規関数,新規ファイルのときは自分流で作っています.大まかなコメントの付け方とかは真似るでしょうけど,括弧の細かい位置とかスペースとタブの使い分けとか空行とかをわざわざ真似しようとは思いません.1つの関数の中で十分に見やすければそれでいいと思いますけど.
                  >
                  > あと,ファイル末尾の/*[EOF]*/は何の意味があるんでしょう?
                  >
                  > 徹底しようとすればコーディング規約を定めて,しかも修正を反映するときに誰かがスタイルチェックを行わないと難しいですが,そこまで労力をかける価値は無いのではと思います.
                  >
                  > 抜本的な変更といえば,関数・クラスのコメント整備というのも全部やった方がいいのかもしれないが,量的に厳しい.
                  >
                  > >ホワイトスペースを無視できない?(前は無視してたハズですが。)
                  > あれは私がファイルを比較して,変更点が見つからなかったので更新するのをやめたのであって,管理ツール自体はスペースを無視しません.
                  • [624] Re5: 意味はあると思うからやった nakatani 2001年07月17日 21:09

                    ファイル末尾の/*[EOF]*/は
                    grepするとファイルの行数がわかるので
                    全ソースの規模がどれぐらいだとか
                    つけてました。ぜんぜんなくてもいいので
                    消してもいいですよ
                    原作のソースの書き方はめちゃくちゃなんで
                    皆さん好きなように書いていいですよ
                    ただわかりやすくメンテナンスや機能追加が
                    簡単にできるように、僕みたいに根気がなくても
                    読めるようなソースを書いてもらえれば
                    うれしいなあ って原作のソースはぐちゃぐちゃですね

                    最近何から手をつけていいのやらわからないので
                    ほとんど作業してません。ソースのバージョンアップを
                    眺めてるだけです がんばるぞー

                    • [625] プログラム規模 げんた 2001年07月18日 09:26

                      ▼ nakataniさん
                      >grepするとファイルの行数がわかるので
                      >全ソースの規模がどれぐらいだとか

                      規模を計るならコメント書うんた↓
                      http://member.nifty.ne.jp/MASE/html/cmtw.htm
    • [613] Re: RC4.3登録 げんた 2001年07月16日 22:05

      ところで,7/16修正のMIME decodeの有効化は?
      • [615] Re2: RC4.3登録 じぇぷろ 2001年07月16日 22:33

        ▼ げんたさん
        > ところで,7/16修正のMIME decodeの有効化は?
        Q内容が掴めませんが、有効化したファイルはどこか?ですか、それとも有効化した理由は?ですか? 単独ファイルでおいてあります。まあ、コメントの通り誰でも修正できる内容です。(これでいいならの話。 厳密なことは知らん―一応これで変換出来ることは確認済み)
        原作版で出来たことが出来なくなっている、と聞いてともかくすぐできそうなことを(誰もやらないので)したまでです。
        • [616] Re3: RC4.3登録 げんた 2001年07月16日 23:15

          >単独ファイルでおいてあります。
          すいません.見落としていました.