0

kiloという1000行ほどで書かれたシンプルなエディタがあります

kilo github page

このコードを読んでいるのですが、551行目の意味がよくわかりません

551        row->render = malloc(row->size + tabs*8 + nonprint*9 + 1);

nonprint*9はnon-printableなcharのために9byteも割り当てています

しかも、543行目でnonprint = 0となっており、この行以外でnonprintの値が操作されることはありません (551行目では常にnonprint = 0となる)

543        int tabs = 0, nonprint = 0, j, idx;

551行目のnonprint*9にはどのような意図があるのでしょうか?

1
  • 3
    READMEファイルにこう記載されています。The project is in alpha stage and was written in just a few hours 「このプロジェクトはまだアルファステージで、ほんの数時間で書かれたものなんです。」つまりまだまだ完成品にはほど遠いということです。世の中には「完成」とされているソフトの中でさえ、書きかけで放置されたか、消しかけで消し残されたコードが残っている場合があります。ましてやアルファ版のソフトに完全な整合性を求めるのは無理です。あなたが確認されているように現在のコードでは同関数内のnonprintはずっと0ですから全く意味を持ちません。「どのような意図があるのでしょうか?」というご質問なので、yohjpさんは同じコードの他の箇所の記述から、元はこのような意図で書かれたもの(の痕跡、名残)ではないかと推測されているわけで、私も妥当な推測だと思います。
    – OOPer
    Commented 2016年9月23日 13:55

1 件の回答 1

1

推測に過ぎませんが、editorRefreshScreen関数ではHL_NONPRINTに対して色変更のエスケープシーケンス(4byte)を2回出力していますから、4 + 1 + 4 = 9 ではないでしょうか?

https://github.com/antirez/kilo/blob/efd541bb6d272f953e049a85d411765eed1a2979/kilo.c#L897-L905

2
  • nonprintは543行目(同一の関数内)で0になります。 そうなるとnonprint*9は必ず0になってしまうのでしょうか? Commented 2016年9月23日 3:09
  • リンク先バージョンを局所的に解釈すれば仰る通りですね。あとはLe Pere d'OOさんコメントの通りです。
    – yohjp
    Commented 2016年9月25日 4:30

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。