syuji-higa/gulp-cr-lf-replace

Line feed code convert Gulp plugin.

gulp-cr-lf-replace

gulp = require 'gulp'
crLfReplace = require 'gulp-cr-lf-replace'

gulp.task 'cr-lf-replace', ->
  gulp
    .src './htdocs/**/*.html'
    .pipe crLfReplace
      changeCode: 'CR+LF' # CR+LF, LF, CR
    .pipe gulp.dest './'

gulp-cr-lf-replace's Issues

複数の改行コードが混在しているファイルで変換漏れが発生する

バグ内容

1ファイルにCRLF, LF, CRのうち2つ以上の改行コードが混在している場合、変換漏れが発生するようです。

再現・確認手順

$ cat crlf.txt lf.txt > cat.txt # 改行コードがCRLF, LFの2つのテキストファイルを結合
$ od -c cat.txt # \r\n と \n が混在していることを確認
$ gulp cr-lf-replace # gulp タスクを実行（たとえばCRLFに統一する）
$ od -c cat.txt # 実行前と変わらず

原因

上記のケースでは、ロジック上、CRLFをCRLFに変換しており、結果なにも変換できていないという状態になっているようです。
元ファイルの改行コードチェックの際にひとつでもCRLFが見つかれば beforeCode がCRLFに決定されてしまうためだと思われます。

修正案

ファイル内の改行コードは1つとは限らないため（上の例では cat で結合しましたが gulp 等でも改行コードを意識せずファイル結合など行うと同じ状態となります）、元ファイルの改行コードのチェックは行わず、無条件で指定の改行コードへ変換するようにしたほうがいいかもしれません。