New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jsonp support, security fix, tiny code refactoring #1

Merged
merged 2 commits into from Mar 24, 2011

Conversation

Projects
None yet
2 participants
@withgod
Contributor

withgod commented Mar 24, 2011

  • 被っちゃいましたが、jsonpのサポート(postもみてます)
  • callback引数を何もチェックしてないのは不味いのでチェック
  • usageのチェックもresult返すところで一括チェック
@sgk

This comment has been minimized.

Show comment
Hide comment
@sgk

sgk Mar 24, 2011

Owner

ありがとうございます。みてみます。

Owner

sgk commented Mar 24, 2011

ありがとうございます。みてみます。

@sgk sgk merged commit 7febc85 into sgk:master Mar 24, 2011

@sgk

This comment has been minimized.

Show comment
Hide comment
@sgk

sgk Mar 24, 2011

Owner

マージして、少々リファクタしました。リストの長さがゼロの時は、その前のif objがFalseになるので、チェック不要と思います。正規表現は、先にコンパイルしておくことで高速化しました。

Owner

sgk commented Mar 24, 2011

マージして、少々リファクタしました。リストの長さがゼロの時は、その前のif objがFalseになるので、チェック不要と思います。正規表現は、先にコンパイルしておくことで高速化しました。

@withgod

This comment has been minimized.

Show comment
Hide comment
@withgod

withgod Mar 24, 2011

Contributor

全然python使ってないので、細かい所はお任せします。
今回もリファレンス引きながら書いたので ;)

Contributor

withgod commented Mar 24, 2011

全然python使ってないので、細かい所はお任せします。
今回もリファレンス引きながら書いたので ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment