やたがわです。 > http://github.com/cvmat/twittering-mode/tree/fix-get-response
異常事態であることを検出できれば私は何を返してもよいと 思いますので、修正することに異論はありません。 ただ、このtopic branchの先端では(message 〜)が追加されて いますが、twittering-get-response-header()の呼び出し元でも messageが呼び出されていて、すぐに上書きされてしまうので、 前者はほとんど視認できないのではないかと思います。 > 松尾です。 > highmtさん、やたがわさん、ご指摘ありがとうございます。 > テスト時に使っていた変数名が残ってしまっていたようです。 > やたがわさんにはsearch-forward-regexp()のNOERRORオプ > ションの件でもお世話になりました。ありがとうございます。 > 今回のやたがわさんの修正ではHTTP headerの終了行が見つから > ない場合にbufferの内容全てを返すようになっていますが、 > 異常事態なので空文字列を返した方が良いんじゃないでしょうか。 > http://github.com/cvmat/twittering-mode/tree/fix-get-response > にcommitしてあります。 > twittering-get-response-body()の方も、終了行が見つからない > 場合にはxml-parse-region()を呼ばずにnilを返すようにして > みました。 > twittering-get-response-body()はwhenを使っても良いのですが > エラー時の返り値を明示する目的でifにしてあります。 > どうでしょうか。 > --- > 松尾 直志 <[email protected]> > ------------------------------------------------------------------------------ > This SF.Net email is sponsored by the Verizon Developer Community > Take advantage of Verizon's best-in-class app development support > A streamlined, 14 day to market process makes app distribution fast and easy > Join now and get one step closer to millions of Verizon customers > http://p.sf.net/sfu/verizon-dev2dev > _______________________________________________ > twmode-users mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/twmode-users ------------------------------------------------------------------------------ This SF.Net email is sponsored by the Verizon Developer Community Take advantage of Verizon's best-in-class app development support A streamlined, 14 day to market process makes app distribution fast and easy Join now and get one step closer to millions of Verizon customers http://p.sf.net/sfu/verizon-dev2dev _______________________________________________ twmode-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/twmode-users
