{"thread":{"id":"19582","subject":"[PATCH 00/17] http*: refactor (v4)","startedAt":"2009-05-29T17:58:28Z","lastAt":"2009-05-29T17:58:28Z","messageCount":1,"participants":["Tay Ray Chuan"],"isPatch":true,"patchVersion":1,"patchTotal":17},"messages":[{"id":"115024","messageId":"20090530015828.2de63b0a.rctay89@gmail.com","threadId":"19582","inReplyTo":null,"subject":"[PATCH 00/17] http*: refactor (v4)","fromName":"Tay Ray Chuan","fromEmail":"rctay89@gmail.com","sentAt":"2009-05-29T17:58:28Z","receivedAt":"2009-05-29T17:58:28Z","isPatch":true,"sender":{"key":"rctay89@gmail.com","avatar":"https://avatars.githubusercontent.com/u/61553?v=4"},"body":"Junio: this replaces rc/http-push~10 in pu. I've included your\nsuggestions and changes.\n\nMike: I've only taken some of your patches from your WIP patch series\ndated Jan 18 (http://kerneltrap.org/mailarchive/git/2009/1/18/4757804).\n\nHere's a list of changes from Mike's patches:\n\n- the contentious http_cleanup has been removed in patch #6 (cleanup).\n\n- a new http_request return state, HTTP_START_FAILED, has been added.\n\n- http_request prints an error message (\"Unable to start request\") when\n  start_active_slot fails.\n\n- a new http API method, http_error, has been added.\n\n- in patch #11 (http_fetch_ref), the error (\"Unable to start request\")\n  is still removed, like Mike did, since http_request already handles \n  it.\n\n- in patch #12 (remote_exists), the patch has been re-titled from\n  update_remote_info_refs.\n\nMike Hommey (7):\n  Don't expect verify_pack() callers to set pack_size\n  transport.c::get_refs_via_curl(): do not leak refs_url\n  http.c: new functions for the http API\n  transport.c::get_refs_via_curl(): use the new http API\n  http.c::http_fetch_ref(): use the new http API\n  http-push.c::remote_exists(): use the new http API\n  http-push.c::fetch_symref(): use the new http API\n\nTay Ray Chuan (10):\n  http-push, http-walker: style fixes\n  http-walker: verify remote packs\n  http*: copy string returned by sha1_to_hex\n  http-push: do not SEGV after fetching a bad pack idx file\n  http*: move common variables and macros to http.[ch]\n  http: create function end_url_with_slash\n  http*: add http_get_info_packs\n  http: use new http API in fetch_index()\n  http*: add helper methods for fetching packs\n  http*: add helper methods for fetching objects (loose)\n\n http-push.c   |  536 ++++++---------------------------------------------\n http-walker.c |  530 ++++++++------------------------------------------\n http.c        |  598 ++++++++++++++++++++++++++++++++++++++++++++++++++++++---\n http.h        |   85 ++++++++\n pack-check.c  |    8 +-\n transport.c   |   33 +--\n 6 files changed, 820 insertions(+), 970 deletions(-)\n\n-- \nCheers,\nRay Chuan\n"}]}