はやみずです > > 異常事態であることを検出できれば私は何を返してもよいと > > 思いますので、修正することに異論はありません。 > > ただ、使わないのであれば呼び出し側で間違えて使うことのない > ようにしておいた方が良いような気がしましたので。 > # 個人的にチェックし忘れることが多いので…。
単純にerrorを投げてしまうのが筋じゃないですかね。headerとbodyの区切りが 見付からない時点でレスポンスとして壊れてしまっているので、errorが自然で しょう。 At Thu, 31 Dec 2009 00:23:28 +0900 (JST), Tadashi MATSUO wrote: > > 松尾です。 > > > 異常事態であることを検出できれば私は何を返してもよいと > > 思いますので、修正することに異論はありません。 > > headerに関しては > (string-match "HTTP/1\.[01] \\([a-zA-Z0-9 ]+\\)\r?\n" header) > のチェックが入っているので、「空行なしでヘッダ部分だけ」や > 「一部にHTTP headerぽい文字列を含むデータ」のようなややこしい > 応答でなければ元々問題ないと思います。 > > ただ、使わないのであれば呼び出し側で間違えて使うことのない > ようにしておいた方が良いような気がしましたので。 > # 個人的にチェックし忘れることが多いので…。 > > > ただ、このtopic branchの先端では(message 〜)が追加されて > > いますが、twittering-get-response-header()の呼び出し元でも > > messageが呼び出されていて、すぐに上書きされてしまうので、 > > 前者はほとんど視認できないのではないかと思います。 > > これは気付いてませんでした。ご指摘ありがとうございます。 > twittering-get-response-body()で(message "Failure: %s" ...)と > なっていたのでそれに合わせたつもりでした。 > > twittering-http-get-list-index-sentinel()を見て出力されない > ものだとばかり思っていたのですが > twittering-other-user-list-interactive()の側で出力されていたの > ですね。*Messages* bufferに記録が残るというのはありますが、 > あまり意味なかったですね…。 > > --- > 松尾 直志 <[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
