はやみずです

> > 異常事態であることを検出できれば私は何を返してもよいと
> > 思いますので、修正することに異論はありません。
>
> ただ、使わないのであれば呼び出し側で間違えて使うことのない
> ようにしておいた方が良いような気がしましたので。
> # 個人的にチェックし忘れることが多いので…。

単純に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に記録が残るというのはありますが、
> あまり意味なかったですね…。
> 
> ---
> 松尾 直志 <t...@mymail.twin.jp>
> 
> ------------------------------------------------------------------------------
> 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
> twmode-users@lists.sourceforge.net
> 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
twmode-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/twmode-users

メールによる返信