suggest

oauth_sign_array2

気が向いたらoauth_sign_url2からoauth_sign_array2に変える． でもそうするとadd_.*の実装が意味なくなって全部書きなおし． しかもargcとargvをいちいちreallocするの面倒

リクエストの引数の付け方は議論の予知あり． しかしlibに引数用の関数があるのだからそれを使ったほうがいいか？

いい実装ないかな

?: http://d.hatena.ne.jp/sugyan/20100816/1281898747

add_()系とfree()

• add_()系はdefineマクロでまとめられるんじゃなイカ？しかもaddじゃなくてappendじゃなイカ？ 変更中．下記参照
• free(p);p=NULL;もマクロでまとめた方がいいじゃなイカ？ やった

ソース分割

http_request()を編集しただけでmakeすると無駄に時間かかるから分割しなイカ？ できればAPIのprefixごとに分割すべきじゃなイカ？

エラー時の返り値

正常終了していないのに0が帰るのはおかしくなイカ？とりあえず，return 001や0001みたいに後で変更できるようなユニークな0以外の値にすべきじゃなイカ？

alloc_strcat

strncatの定義が

#include <string.h>
char *strncat(char * restrict s1, const char * restrict s2, size_t n);

となっているんだからalloc_strcatもrestrict修飾子をつけるべきじゃなイカ？

とりあえずwarning出すようにした．

enum vs const

現状ヘッダにenumで定数を定義しているが，ソースファイルにconstで定義して，externする方法も考えられる．

add_argsマクロ

関数の名前は違えども引数の型は同じで，同じ動作をする関数群が大量にある．これらを一つにまとめられるのでは． また，引数を変更しないのだからconstをつけるべき．同様に公開する関数の方にもconstをつけるべき．