[前][次][番号順一覧][スレッド一覧]

rabbit-shockers:104

From: MIYAMUKO Katsuyuki <miyamuko@m...>
Date: Wed, 08 Feb 2006 23:43:15 +0900
Subject: [rabbit-shockers:104] Re: リリース予告

みやむこです。

Kouhei Sutou wrote:
> 了解しました.
> # Gdk::Pixbuf#saveのところがgdk_pixbuf_savevのところに
> #   utf8_filename = g_locale_to_utf8 (filename, -1, NULL, NULL, error);
> # というコードがあるのになんでUTF8->SJISな変換をしているのか
> # が納得できていませんが,実際にそう動いているのならそうなの
> # でしょう.
あ、そうなってるんですか。不思議ですねぇ。
#私が勘違いしているだけかも^^

>>かなり quick hack ぎみですが、以下で直りました。
> コミットしてもらえますか?

> 気持ちの問題ですが,ここに
>              file_name_format = GLib.filename_to_utf8(filename_format)
> をいれてもらえますか?
そのようにしました。

> # コメントじゃなくてコードで示すのは私の趣味です.
私もです :-)。1行コメントを見るとリファクタリングしたくなる病ですね。

> Shift_JIS固定じゃなくてどうにかしたいところですが,まぁ,今
> 回はこれでいきましょう.r-tでアナウンスする気力もないですし.
> どうせ,template.erbの中でlang="ja" xml:lang="ja"とか書いちゃっ
> てるし...

すいません。寝ぼけていました。ブラウザの文字コードまで変える必要ないですね。。。
最終的には以下のようにしてコミットしました。

Index: generator.rb
===================================================================
--- generator.rb	(revision 561)
+++ generator.rb	(working copy)
@@ -32,6 +32,10 @@

       def save(file_name_format, slide_number, image_type)
         file_name = slide_file_name(file_name_format, slide_number)
+        if Utils.windows?
+          file_name = GLib.filename_to_utf8(file_name)
+          file_name = GLib.locale_from_utf8(file_name)
+        end
         File.open(file_name, "w") do |f|
           f.print(to_html(file_name_format, slide_number, image_type))
         end


--
ML: rabbit-shockers@m...
使い方: http://QuickML.com/

[前][次][番号順一覧][スレッド一覧]

        91 2006-02-08 10:42 [kou@c...            ] リリース予告                            
        96 2006-02-08 17:20 ┣[miyamuko@m...       ]                                       
        97 2006-02-08 17:51 ┃┗[kou@c...            ]                                     
   @    98 2006-02-08 18:21 ┃ ┗[kou@c...            ]                                   
        99 2006-02-08 18:55 ┃  ┗[miyamuko@m...       ]                                 
       101 2006-02-08 21:43 ┃   ┗[kou@c...            ]                               
->     104 2006-02-08 23:43 ┃    ┗[miyamuko@m...       ]                             
       105 2006-02-08 23:53 ┃     ┗[kou@c...            ]                           
       107 2006-02-09 02:00 ┃      ┗[miyamuko@m...       ]                         
       100 2006-02-08 19:29 ┗[miyamuko@m...       ]                                       
       102 2006-02-08 21:45  ┗[kou@c...            ]