Pierre Habouzit [off-list ref] writes:
quoted hunk
If the gain is not obvious in the diffstat, the resulting code is more
readable, _and_ in checkout-index/update-index we now reuse the same buffer
to unquote strings instead of always freeing/mallocing.
This also is more coherent with the next patch that reworks quoting
functions.
The quoting function is also made more efficient scanning for backslashes
and treating portions of strings without a backslash at once.
Signed-off-by: Pierre Habouzit <redacted>
---
builtin-apply.c | 125 +++++++++++++++++++++++-----------------------
builtin-checkout-index.c | 27 +++++-----
builtin-update-index.c | 51 ++++++++++---------
fast-import.c | 47 ++++++++---------
mktree.c | 25 +++++----
quote.c | 92 ++++++++++++++++------------------
quote.h | 2 +-
7 files changed, 184 insertions(+), 185 deletions(-)
...
diff --git a/quote.c b/quote.c
index 4df3262..67c6527 100644
--- a/quote.c
+++ b/quote.c
@@ -201,68 +201,62 @@ int quote_c_style(const char *name, char *outbuf, FILE *outfp, int no_dq)
* should free when done. Updates endp pointer to point at
* one past the ending double quote if given.
*/
You need to update the comment above which talks about the input
and return values. You no longer return an allocated memory
which the caller should free. You return something else.
On Wed, Sep 19, 2007 at 08:09:19AM +0000, Junio C Hamano wrote:
Pierre Habouzit [off-list ref] writes:
quoted
If the gain is not obvious in the diffstat, the resulting code is more
readable, _and_ in checkout-index/update-index we now reuse the same buffer
to unquote strings instead of always freeing/mallocing.
This also is more coherent with the next patch that reworks quoting
functions.
The quoting function is also made more efficient scanning for backslashes
and treating portions of strings without a backslash at once.
Signed-off-by: Pierre Habouzit <redacted>
---
builtin-apply.c | 125 +++++++++++++++++++++++-----------------------
builtin-checkout-index.c | 27 +++++-----
builtin-update-index.c | 51 ++++++++++---------
fast-import.c | 47 ++++++++---------
mktree.c | 25 +++++----
quote.c | 92 ++++++++++++++++------------------
quote.h | 2 +-
7 files changed, 184 insertions(+), 185 deletions(-)
...
diff --git a/quote.c b/quote.c
index 4df3262..67c6527 100644
--- a/quote.c
+++ b/quote.c
@@ -201,68 +201,62 @@ int quote_c_style(const char *name, char *outbuf, FILE *outfp, int no_dq)
* should free when done. Updates endp pointer to point at
* one past the ending double quote if given.
*/
You need to update the comment above which talks about the input
and return values. You no longer return an allocated memory
which the caller should free. You return something else.
Oh my, okay I'll do that. I intend to send an updated series at some
point anyways, because of the two mistakes I already spotted, and
because next conflicts with this series (in not too hard ways to deal
with but still, that would save you some useless merges).
--
·O· Pierre Habouzit
··O madcoder@debian.org
OOO http://www.madism.org