Gauche > Archives > 2013/06/01

2013/06/01 00:57:10 UTCenami
#
なるほど、言われてみれば一行読むとかそれ自体オーバーヘッドですね。ポートでバッファリングしているんだし。
2013/06/01 01:42:02 UTCyamasushi
#
つまり、文字列として処理するよりもジェネレータとして処理したほうがいい場合だったということですか?
2013/06/01 04:46:58 UTCyamasushi
#
あ、ジェネレータというのは違うか。文字列として読むのでなく、バッファにあるものを一つ一つとっていく=ジェネレータと短絡してしまいました。
2013/06/01 06:00:57 UTCyamasushi
#
修正前のcsvの実装を読もうとして、よくわからなくて挫折した記憶があります。そのときには性能を出すためにはこういうテクニックがいるのかなあと思っていたのですが・・・・・・事実は小説より奇なりというか面白いものですね。
2013/06/01 06:17:03 UTCenami
#
skip-ws は改行でも止まる必要がありますね。でないと ,\na,b\n のような場合に次の行まで一つのレコードになってしまいます。
#
diff --git a/lib/text/csv.scm b/lib/text/csv.scm
index 2f4f037..cb1f385 100644
--- a/lib/text/csv.scm
+++ b/lib/text/csv.scm
@@ -81,10 +81,11 @@
   (define (eor? ch) (or (eof-object? ch) (eqv? ch #\newline)))
 
   (define (start fields)
-    (let1 ch (skip-ws (read-char))
+    (let1 ch (read-char)
       (cond [(eor? ch) (reverse (cons "" fields))]
             [(eqv? ch sep) (start (cons "" fields))]
             [(eqv? ch quo) (quoted fields)]
+            [(char-whitespace? ch) (start fields)]
             [else (unquoted (list ch) fields)])))
 
   (define (unquoted chs fields)
#
skip-ws なくしてこうでいいような。
#
あと eor? のテストをひっくり返すとこの間の郵便番号の csv ファイルで 2-3% 速くなるようです。
2013/06/01 06:36:00 UTCshiro
#
まはろ。コミットしました。
2013/06/01 21:48:40 UTCkaki
#
gosh> #/((.)\2)/
*** ERROR: bad regexp syntax in "((.)\\2)"
#
バグっぽいです
2013/06/01 23:22:01 UTCshiro
#
む。regexp.cの944行目 if (ctx->grpcount - level < grpno) goto synerr; なんだけどなんでわざわざlevelを引いてるんだっけな。
2013/06/01 23:29:11 UTCshiro
#
自分を含むグループの参照を禁止しようとした? でもlevelじゃそれは実現できないな。どのグループが閉じたかの情報を別に持っとかないとだめなんじゃないかな。