Re: [PATCH 1/4] http.h: Add debug callback and helper routine for implementing the GIT_TRACE_CURL environment variable in http.c
On 19/04/16 22:48, Ramsay Jones wrote: > [snip] > I think the minimal fixup (including Junio's comment on patch #2, which also > triggered for me) is given in the patch below. BTW, if you want to have a single static instance of the 'struct trace_key', then the following patch on top should work. (Also, I have compiled and run the testsuite on these patches, but _not_ tested that the trace functionality actually works!) ;-) HTH ATB, Ramsay Jones -- >8 -- Subject: [PATCH] curl-trace: reduce visibility of the trace key struct Signed-off-by: Ramsay Jones--- http.c | 9 +++-- http.h | 2 +- imap-send.c | 2 +- 3 files changed, 9 insertions(+), 4 deletions(-) diff --git a/http.c b/http.c index ce91421..eed6dba 100644 --- a/http.c +++ b/http.c @@ -11,7 +11,7 @@ #include "gettext.h" #include "transport.h" -struct trace_key trace_curl = TRACE_KEY_INIT(CURL); +static struct trace_key trace_curl = TRACE_KEY_INIT(CURL); #if LIBCURL_VERSION_NUM >= 0x070a08 long int git_curl_ipresolve = CURL_IPRESOLVE_WHATEVER; #else @@ -516,6 +516,11 @@ static void curl_dump(const char *text, unsigned char *ptr, size_t size, char no } } +int curl_trace_want(void) +{ + return trace_want(_curl); +} + int curl_trace(CURL *handle, curl_infotype type, char *data, size_t size, void *userp) { @@ -652,7 +657,7 @@ static CURL *get_curl_handle(void) "your curl version is too old (>= 7.19.4)"); #endif - if (trace_want(_curl)) { + if (curl_trace_want()) { curl_easy_setopt(result, CURLOPT_VERBOSE, 1L); curl_easy_setopt(result, CURLOPT_DEBUGFUNCTION, curl_trace); curl_easy_setopt(result, CURLOPT_DEBUGDATA, NULL); diff --git a/http.h b/http.h index 00e4ad7..153ff17 100644 --- a/http.h +++ b/http.h @@ -225,7 +225,7 @@ extern void abort_http_object_request(struct http_object_request *freq); extern void release_http_object_request(struct http_object_request *freq); /* Debug callback and helper routine for curl_easy_setopt CURLOPT_DEBUGFUNCTION */ -extern struct trace_key trace_curl; +int curl_trace_want(void); int curl_trace(CURL *handle, curl_infotype type, char *data, size_t size, void *userp); diff --git a/imap-send.c b/imap-send.c index 36ff843..627ffa4 100644 --- a/imap-send.c +++ b/imap-send.c @@ -1444,7 +1444,7 @@ static CURL *setup_curl(struct imap_server_conf *srvc) if (0 < verbosity || getenv("GIT_CURL_VERBOSE")) curl_easy_setopt(curl, CURLOPT_VERBOSE, 1L); - if (trace_want(_curl)) { + if (curl_trace_want()) { curl_easy_setopt(curl, CURLOPT_VERBOSE, 1L); curl_easy_setopt(curl, CURLOPT_DEBUGFUNCTION, curl_trace); curl_easy_setopt(curl, CURLOPT_DEBUGDATA, NULL); -- 2.8.0 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/4] http.h: Add debug callback and helper routine for implementing the GIT_TRACE_CURL environment variable in http.c
On 19/04/16 16:10, Elia Pinto wrote: > Add the debug callback and helper routine prototype used by > curl_easy_setopt CURLOPT_DEBUGFUNCTION in http.c > for implementing the GIT_TRACE_CURL environment variable > > > Helped-by: Torsten Bögershausen> Helped-by: Ramsay Jones > Helped-by: Junio C Hamano > Helped-by: Eric Sunshine > Helped-by: Jeff King > Signed-off-by: Elia Pinto > --- > http.h | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/http.h b/http.h > index 4ef4bbd..a2d10bc 100644 > --- a/http.h > +++ b/http.h > @@ -224,4 +224,10 @@ extern int finish_http_object_request(struct > http_object_request *freq); > extern void abort_http_object_request(struct http_object_request *freq); > extern void release_http_object_request(struct http_object_request *freq); > > +/* Debug callback and helper routine for curl_easy_setopt > CURLOPT_DEBUGFUNCTION */ > +static struct trace_key trace_curl = TRACE_KEY_INIT(CURL); Ah no, this would add 6 instances of the 'trace_curl' key in http-fetch.c, http-push.c, http-walker.c, http.c, imap-send.c and remote-curl.c. Hmm ... since these would end up in different executables (by and large) it might work OK, ... but is simply not necessary. Also, patches #1 and #2 should be squashed into one patch and, since the curl_dump() function is only called from http.c, it can be a static symbol. I think the minimal fixup (including Junio's comment on patch #2, which also triggered for me) is given in the patch below. Hope that helps. ATB, Ramsay Jones > +int curl_trace(CURL *handle, curl_infotype type, char *data, size_t size, > void *userp); > +void curl_dump(const char *text, unsigned char *ptr, size_t size, char > nohex); > + > + > #endif /* HTTP_H */ -- >8 -- Subject: [PATCH] curl-trace: fix scope/visibility of various symbols Signed-off-by: Ramsay Jones --- http.c | 9 +++-- http.h | 3 +-- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/http.c b/http.c index 64dd975..ce91421 100644 --- a/http.c +++ b/http.c @@ -11,9 +11,7 @@ #include "gettext.h" #include "transport.h" -/* -tatic struct trace_key trace_curl = TRACE_KEY_INIT(CURL); -*/ +struct trace_key trace_curl = TRACE_KEY_INIT(CURL); #if LIBCURL_VERSION_NUM >= 0x070a08 long int git_curl_ipresolve = CURL_IPRESOLVE_WHATEVER; #else @@ -468,12 +466,11 @@ static void set_curl_keepalive(CURL *c) #endif -void curl_dump(const char *text, unsigned char *ptr, size_t size, char nohex) +static void curl_dump(const char *text, unsigned char *ptr, size_t size, char nohex) { size_t i; size_t w; - struct strbuf out = STRBUF_INIT;; - + struct strbuf out = STRBUF_INIT; unsigned int width = 0x10; if (nohex) diff --git a/http.h b/http.h index a2d10bc..00e4ad7 100644 --- a/http.h +++ b/http.h @@ -225,9 +225,8 @@ extern void abort_http_object_request(struct http_object_request *freq); extern void release_http_object_request(struct http_object_request *freq); /* Debug callback and helper routine for curl_easy_setopt CURLOPT_DEBUGFUNCTION */ -static struct trace_key trace_curl = TRACE_KEY_INIT(CURL); +extern struct trace_key trace_curl; int curl_trace(CURL *handle, curl_infotype type, char *data, size_t size, void *userp); -void curl_dump(const char *text, unsigned char *ptr, size_t size, char nohex); #endif /* HTTP_H */ -- 2.8.0 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/4] http.h: Add debug callback and helper routine for implementing the GIT_TRACE_CURL environment variable in http.c
Elia Pintowrites: > Add the debug callback and helper routine prototype used by > curl_easy_setopt CURLOPT_DEBUGFUNCTION in http.c > for implementing the GIT_TRACE_CURL environment variable > > > Helped-by: Torsten Bögershausen > Helped-by: Ramsay Jones > Helped-by: Junio C Hamano > Helped-by: Eric Sunshine > Helped-by: Jeff King > Signed-off-by: Elia Pinto > --- > http.h | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/http.h b/http.h > index 4ef4bbd..a2d10bc 100644 > --- a/http.h > +++ b/http.h > @@ -224,4 +224,10 @@ extern int finish_http_object_request(struct > http_object_request *freq); > extern void abort_http_object_request(struct http_object_request *freq); > extern void release_http_object_request(struct http_object_request *freq); > > +/* Debug callback and helper routine for curl_easy_setopt > CURLOPT_DEBUGFUNCTION */ > +static struct trace_key trace_curl = TRACE_KEY_INIT(CURL); This will be multiply instanciated in every *.c file that includes http.h? > +int curl_trace(CURL *handle, curl_infotype type, char *data, size_t size, > void *userp); > +void curl_dump(const char *text, unsigned char *ptr, size_t size, char > nohex); > + > + > #endif /* HTTP_H */ In any case, you'd want to combine this with 2/4 and write a single log message for the combined change. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 1/4] http.h: Add debug callback and helper routine for implementing the GIT_TRACE_CURL environment variable in http.c
Add the debug callback and helper routine prototype used by curl_easy_setopt CURLOPT_DEBUGFUNCTION in http.c for implementing the GIT_TRACE_CURL environment variable Helped-by: Torsten BögershausenHelped-by: Ramsay Jones Helped-by: Junio C Hamano Helped-by: Eric Sunshine Helped-by: Jeff King Signed-off-by: Elia Pinto --- http.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/http.h b/http.h index 4ef4bbd..a2d10bc 100644 --- a/http.h +++ b/http.h @@ -224,4 +224,10 @@ extern int finish_http_object_request(struct http_object_request *freq); extern void abort_http_object_request(struct http_object_request *freq); extern void release_http_object_request(struct http_object_request *freq); +/* Debug callback and helper routine for curl_easy_setopt CURLOPT_DEBUGFUNCTION */ +static struct trace_key trace_curl = TRACE_KEY_INIT(CURL); +int curl_trace(CURL *handle, curl_infotype type, char *data, size_t size, void *userp); +void curl_dump(const char *text, unsigned char *ptr, size_t size, char nohex); + + #endif /* HTTP_H */ -- 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html