Thread (9 messages) flat view 9 messages, 2 authors, 2016-06-15
STALE3732d

Revision v2 of 2 in this series.

Revisions (2)
  1. v1 [diff vs current]
  2. v2 current

[PATCH v2 0/7] http: minor improvements

From: Tay Ray Chuan <hidden>
Date: 2016-06-15 22:48:22

This patch series was generated on top of 'next'. It contains some
general changes to http usage.

Some highlights:
  - add non-ff test cases to "dumb" http push test
  - separate init and cleanup of http from http-walker

Changes from v1 (based on comments from Clemens):
  - patch 3: don't say "remove condition", since we're actually enabling
    it.
  - patch 5: in keeping with separating http and http-walker, removed
    http_cleanup() from http-walker->cleanup().
  - patch 5: move http_cleanup() call after http-walker->cleanup(). This
    isn't really needed, but do this just in case http-walker->cleanup()
    does any http operations.

Contents:

[PATCH v2 1/7] t5541-http-push: check that ref is unchanged for non-ff test
[PATCH v2 2/7] t554[01]-http-push: refactor, add non-ff tests
[PATCH v2 3/7] http-push: remove "|| 1" to enable verbose check
[PATCH v2 4/7] http-walker: cleanup more thoroughly
[PATCH v2 5/7] http: init and cleanup separately from http-walker
[PATCH v2 6/7] remote-curl: use http_fetch_ref() instead of walker wrapper
[PATCH v2 7/7] remote-curl: init walker only when needed

 http-fetch.c         |    5 ++++-
 http-push.c          |    2 +-
 http-walker.c        |   21 +++++++++++++++++----
 remote-curl.c        |   21 +++++++++------------
 t/lib-httpd.sh       |   29 +++++++++++++++++++++++++++++
 t/t5540-http-push.sh |    3 +++
 t/t5541-http-push.sh |   22 ++--------------------
 walker.h             |    2 +-
 8 files changed, 66 insertions(+), 39 deletions(-)

--
Cheers,
Ray Chuan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help