* [PATCH] protocol: make parse_protocol_version() private
@ 2017-09-17 16:53 Ramsay Jones
2017-09-18 16:38 ` Brandon Williams
0 siblings, 1 reply; 2+ messages in thread
From: Ramsay Jones @ 2017-09-17 16:53 UTC (permalink / raw)
To: Brandon Williams; +Cc: Junio C Hamano, GIT Mailing-list
Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
---
Hi Brandon,
If you need to re-roll your 'bw/protocol-v1' branch, could you please
squash this into the relevant patch (commit 45954f179e, "protocol:
introduce protocol extention mechanisms", 13-09-2017).
This assumes you agree that this symbol does not need to be public; if
not, then please just ignore! ;-)
Thanks!
ATB,
Ramsay Jones
protocol.c | 2 +-
protocol.h | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/protocol.c b/protocol.c
index 1b16c7b9a..369503065 100644
--- a/protocol.c
+++ b/protocol.c
@@ -2,7 +2,7 @@
#include "config.h"
#include "protocol.h"
-enum protocol_version parse_protocol_version(const char *value)
+static enum protocol_version parse_protocol_version(const char *value)
{
if (!strcmp(value, "0"))
return protocol_v0;
diff --git a/protocol.h b/protocol.h
index 2fa6486d0..18f9a5235 100644
--- a/protocol.h
+++ b/protocol.h
@@ -7,7 +7,6 @@ enum protocol_version {
protocol_v1 = 1,
};
-extern enum protocol_version parse_protocol_version(const char *value);
extern enum protocol_version get_protocol_version_config(void);
extern enum protocol_version determine_protocol_version_server(void);
extern enum protocol_version determine_protocol_version_client(const char *server_response);
--
2.14.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] protocol: make parse_protocol_version() private
2017-09-17 16:53 [PATCH] protocol: make parse_protocol_version() private Ramsay Jones
@ 2017-09-18 16:38 ` Brandon Williams
0 siblings, 0 replies; 2+ messages in thread
From: Brandon Williams @ 2017-09-18 16:38 UTC (permalink / raw)
To: Ramsay Jones; +Cc: Junio C Hamano, GIT Mailing-list
On 09/17, Ramsay Jones wrote:
>
> Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
> ---
>
> Hi Brandon,
>
> If you need to re-roll your 'bw/protocol-v1' branch, could you please
> squash this into the relevant patch (commit 45954f179e, "protocol:
> introduce protocol extention mechanisms", 13-09-2017).
>
> This assumes you agree that this symbol does not need to be public; if
> not, then please just ignore! ;-)
Thanks! I've updated my local version of the series to reflect this.
>
> Thanks!
>
> ATB,
> Ramsay Jones
>
> protocol.c | 2 +-
> protocol.h | 1 -
> 2 files changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/protocol.c b/protocol.c
> index 1b16c7b9a..369503065 100644
> --- a/protocol.c
> +++ b/protocol.c
> @@ -2,7 +2,7 @@
> #include "config.h"
> #include "protocol.h"
>
> -enum protocol_version parse_protocol_version(const char *value)
> +static enum protocol_version parse_protocol_version(const char *value)
> {
> if (!strcmp(value, "0"))
> return protocol_v0;
> diff --git a/protocol.h b/protocol.h
> index 2fa6486d0..18f9a5235 100644
> --- a/protocol.h
> +++ b/protocol.h
> @@ -7,7 +7,6 @@ enum protocol_version {
> protocol_v1 = 1,
> };
>
> -extern enum protocol_version parse_protocol_version(const char *value);
> extern enum protocol_version get_protocol_version_config(void);
> extern enum protocol_version determine_protocol_version_server(void);
> extern enum protocol_version determine_protocol_version_client(const char *server_response);
> --
> 2.14.0
--
Brandon Williams
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-09-18 16:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-17 16:53 [PATCH] protocol: make parse_protocol_version() private Ramsay Jones
2017-09-18 16:38 ` Brandon Williams
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).