{"thread":{"id":"19679","subject":"[PATCH 00/23] http*: refactor (v5)","startedAt":"2009-06-05T15:59:59Z","lastAt":"2009-06-06T01:12:13Z","messageCount":3,"participants":["Tay Ray Chuan","Jakub Narebski"],"isPatch":true,"patchVersion":1,"patchTotal":23},"messages":[{"id":"115515","messageId":"20090605235959.f4df6457.rctay89@gmail.com","threadId":"19679","inReplyTo":null,"subject":"[PATCH 00/23] http*: refactor (v5)","fromName":"Tay Ray Chuan","fromEmail":"rctay89@gmail.com","sentAt":"2009-06-05T15:59:59Z","receivedAt":"2009-06-05T15:59:59Z","isPatch":true,"sender":{"key":"rctay89@gmail.com","avatar":"https://avatars.githubusercontent.com/u/61553?v=4"},"body":"Junio, this replaces the entire rc/http-push topic.\n\nChanges from the previous patch series:\n\n- fix return values of fetch_indices() in http-push.c, as highlighted\n  by Clemens (patch #20)\n\n- added set slot->local to NULL patch (patch #3)\n\n- set slot->local to NULL in http.c::http_request() (patch #15)\n\n- set slot->local to NULL in http.c::release_http_pack_request() and\n  http.c::finish_http_pack_request() (patch #22)\n\n\nChristian Couder (1):\n  http-push: fix missing \"#ifdef USE_CURL_MULTI\" around\n    \"is_running_queue\"\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 (15):\n  t5540-http-push: test fetching of loose objects\n  t5540-http-push: test fetching of packed objects\n  http*: cleanup slot->local after fclose\n  http-push: send out fetch requests on queue\n  t5550-http-fetch: test fetching of packed objects\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           |  579 ++++++++---------------------------------------\n http-walker.c         |  533 +++++++------------------------------------\n http.c                |  601 +++++++++++++++++++++++++++++++++++++++++++++++--\n http.h                |   85 +++++++\n pack-check.c          |    8 +-\n t/t5540-http-push.sh  |   36 +++\n t/t5550-http-fetch.sh |    8 +\n transport.c           |   33 +--\n 8 files changed, 902 insertions(+), 981 deletions(-)\n"},{"id":"115561","messageId":"m3zlcm5s7g.fsf@localhost.localdomain","threadId":"19679","inReplyTo":"20090605235959.f4df6457.rctay89@gmail.com","subject":"Re: [PATCH 00/23] http*: refactor (v5)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2009-06-05T20:41:30Z","receivedAt":"2009-06-05T20:41:30Z","isPatch":true,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Could you please make all patches in this series to be replies to this\ncover letter ('shallow' threading)?  This way it would be easier to\nread this patch series (or kill / ignore it as a whole)?\n\nThanks in advance\n\nP.S. Why subject is \"Re: [PATCH 00/23] ...\" and not \"[PATCH 00/23] ...\"?\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"115590","messageId":"be6fef0d0906051812j241e78feo1df31d0d244f1b4f@mail.gmail.com","threadId":"19679","inReplyTo":"m3zlcm5s7g.fsf@localhost.localdomain","subject":"Re: [PATCH 00/23] http*: refactor (v5)","fromName":"Tay Ray Chuan","fromEmail":"rctay89@gmail.com","sentAt":"2009-06-06T01:12:13Z","receivedAt":"2009-06-06T01:12:13Z","isPatch":true,"sender":{"key":"rctay89@gmail.com","avatar":"https://avatars.githubusercontent.com/u/61553?v=4"},"body":"Hi,\n\nOn Sat, Jun 6, 2009 at 4:41 AM, Jakub Narebski<jnareb@gmail.com> wrote:\n> Could you please make all patches in this series to be replies to this\n> cover letter ('shallow' threading)?  This way it would be easier to\n> read this patch series (or kill / ignore it as a whole)?\n>\n> Thanks in advance\n\nI'll keep that in mind.\n\n> P.S. Why subject is \"Re: [PATCH 00/23] ...\" and not \"[PATCH 00/23] ...\"?\n\nHmm, I've checked gmane and elsewhere, the subject says \"[PATCH 00/23]...\"...\n\n-- \nCheers,\nRay Chuan\n"}]}