C：crlfをlfに変換したい

Question

表題の通り、文字列リテラルに含まれているcrlfをlfに変換したいのですが、改行が余計に入ってしまう状態です。

void replace_crlf_to_lf(char *buf, size_t size)
{
        for (int n = 0; n < size - 1; n++){
                if (*(buf + n) == '\r'){
                        *(buf + n) = '\n';
                        n += 1;
                }
        }
}

上記のような関数を作ってみたのですが、これを通してみると改行が余計に一列増えてしまう状態です。例えば下記のような形になります。

〇変換前
ABCDEFG
ABCDEFG

〇変換後
ABCDEFG

ABCDEFG

形式としては変換前のように改行を一行挟まずにしたまま各行末尾の改行コードを変換したいのですが、どのようにすればよいのでしょうか。
初歩的な質問で恐縮ですが、ご教示いただけますと幸いです。
よろしくお願いいたします。

Comments

[dodox86]

細かいですが、
> if (*(buf + n) == '\r'){
> *(buf + n) = '\n';

このコードですと、CRLF("\r\n")ではなくて、単にCRだけでもLFに変換してしまいますが良いのでしょうか。

[Ykkykk]

あまり深く改行コードについて理解しておらず、上記のようにしてみておりました。CRだけの場合もあるということですね。ありがとうございます。

[dodox86]

> CRだけの場合もあるということですね。

昔のmacではCRのみが改行でした。まぁ、現代においてはほぼ無いですが、通信データなどに名残りもあるので、100％無い、というわけでもないです。nkfと言うコマンドの改行コード指定オプションにもLFのみ、CRのみ、CRLFなどと言うものがあります。

[Ykkykk]

なるほど。。。
Windowsのメモ帳で作成したテキストファイルをLinux上で開くと改行コードが合わないようだったので、上記のような関数を作成しようかと思った次第でした。環境によっては改行コード自体がそもそも異なる可能性もあるのですね。。

Answer 1

CR+LFだったものをLFにしたいのに、CRをLFに置き換えているだけですよね。
これでは結果はLF+LFとなり２度改行してしまいます。
こんな感じでしょうか？

（追記）
replace_crlf_to_lf関数をvoid型からsize_t型に変更して、変換後のサイズを返した方が
呼び元で使い道があるかと。

size_t replace_crlf_to_lf(char *buf, size_t size)  /* void型からsize_t型に変更 */
{
    for (int n = 0; n < size - 1; n++) {
        if (memcmp(buf + n, "\r\n", 2) == 0) {
            (void)memmove(buf + n, buf + n + 1, size - n - 1);
            size--;
        }
    }
    return size;                                   /* 変換後のサイズを返却 */
}

Comments

[Ykkykk]

ご回答いただきありがとうございます。私もそう思い、二つ目の改行を\0にしてみたのですが、そうすると、それ以降の文字列すべてが消えてしまい、どのように置き換えるのかわからなかったです。。

[DreamTheater]

LFをNULL(\0)にしてしまうと、文字列としては終端とみなされてしまいます。
データとして消えたのではありません。
回答に追記します。

[Ykkykk]

memcmpやmemmoveという関数があるのですね。。。大変勉強になります。
また、呼び出し元での使い道まで考えていただきありがとうございます。このような形で実装したほうがよいですね。ありがとうございます！