From: "Randall S. Becker" <rsbecker@nexbridge.com>
To: "'Junio C Hamano'" <gitster@pobox.com>
Cc: <git@vger.kernel.org>
Subject: RE: [Patch v1 3/3] strbuf.h: remove declaration of deprecated strbuf_write_fd method.
Date: Fri, 19 Jun 2020 15:34:52 -0400 [thread overview]
Message-ID: <02c101d64670$b72ab840$258028c0$@nexbridge.com> (raw)
In-Reply-To: <xmqqa70yhmxz.fsf@gitster.c.googlers.com>
On June 19, 2020 3:32 PM, Junio C Hamano wrote:
> randall.s.becker@rogers.com writes:
>
> > From: "Randall S. Becker" <rsbecker@nexbridge.com>
> >
> > Signed-off-by: Randall S. Becker <rsbecker@nexbridge.com>
> > ---
> > strbuf.h | 1 -
> > 1 file changed, 1 deletion(-)
>
> I think this should be part of 2/3 (otherwise we'd have a decl that nobody
> references that declares a function that nobody implements).
If I understand, combined the strbuf.c and strbuf.h modification into a
single commit, correct? I normally would do that but missed this part of the
contribution standard. If so, I will create v2 accordingly.
>
> > diff --git a/strbuf.h b/strbuf.h
> > index 7062eb6410..223ee2094a 100644
> > --- a/strbuf.h
> > +++ b/strbuf.h
> > @@ -473,7 +473,6 @@ int strbuf_readlink(struct strbuf *sb, const char
> *path, size_t hint);
> > * NUL bytes.
> > */
> > ssize_t strbuf_write(struct strbuf *sb, FILE *stream); -ssize_t
> > strbuf_write_fd(struct strbuf *sb, int fd);
> >
> > /**
> > * Read a line from a FILE *, overwriting the existing contents of
prev parent reply other threads:[~2020-06-19 19:35 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20200619150445.4380-1-randall.s.becker.ref@rogers.com>
2020-06-19 15:04 ` [Patch v1 0/3] Replace strbuf_write_fd with write_in_full randall.s.becker
2020-06-19 15:04 ` [Patch v1 1/3] bugreport.c: replace " randall.s.becker
2020-06-19 16:35 ` Đoàn Trần Công Danh
2020-06-19 17:17 ` Randall S. Becker
2020-06-19 19:30 ` Junio C Hamano
2020-06-19 19:37 ` Randall S. Becker
2020-06-19 23:01 ` Đoàn Trần Công Danh
2020-06-19 23:26 ` Randall S. Becker
2020-06-19 19:47 ` Jeff King
2020-06-19 15:04 ` [Patch v1 2/3] strbuf.c: remove unreferenced strbuf_write_fd method randall.s.becker
2020-06-19 19:30 ` Junio C Hamano
2020-06-19 19:49 ` Jeff King
2020-06-19 15:04 ` [Patch v1 3/3] strbuf.h: remove declaration of deprecated " randall.s.becker
2020-06-19 19:31 ` Junio C Hamano
2020-06-19 19:34 ` Randall S. Becker [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='02c101d64670$b72ab840$258028c0$@nexbridge.com' \
--to=rsbecker@nexbridge.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.