ures that will trigger the fallback, but the later
is likely to need library changes which might not be possible with the old
version anyway.
[1] https://bugs.exim.org/show_bug.cgi?id=1749
Carlo Marcelo Arenas Belón (2):
grep: fallback to interpreter if JIT fails with pcre1
grep: fallback to
starting with 10.23, and as a side effect of the work for bug1749[1] (grep
-P crash with seLinux), pcre2grep was modified to ignore any errors from
pcre2_jit_compile so the interpreter could be used as a fallback
[1] https://bugs.exim.org/show_bug.cgi?id=1749
Signed-off-by: Carlo Marcelo Arenas
to the user.
Signed-off-by: Carlo Marcelo Arenas Belón
---
Makefile | 12 ++--
grep.c | 6 ++
2 files changed, 12 insertions(+), 6 deletions(-)
diff --git a/Makefile b/Makefile
index 1a44c811aa..62b0cb6ee6 100644
--- a/Makefile
+++ b/Makefile
@@ -32,14 +32,14 @@ all
In function 'finish_request',
inlined from 'process_response' at http-push.c:248:2:
http-push.c:587:4: warning: '%s' directive argument is null [-Wformat-overflow=]
587 |fprintf(stderr, "Unable to get pack file %s\n%s",
|^
588 |
se_request unconditionally
at the end.
Signed-off-by: Carlo Marcelo Arenas Belón
Suggested-by: Eric Sunshine
---
http-push.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/http-push.c b/http-push.c
index f675a96316..e36561a6db 100644
--- a/http-push.c
+++ b/http-push.c
883e248b8a ("fsmonitor: teach git to optionally utilize a file system
monitor to speed up detecting new or changed files.", 2017-09-22) uses
an int in a loop that would wrap if index_state->cache_nr (unsigned)
is bigger than INT_MAX
Signed-off-by: Carlo Marcelo Arenas Belón
---
fs
e cast to uint64_t so it also works in 32-bit platforms.
Signed-off-by: Carlo Marcelo Arenas Belón
---
wrapper.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/wrapper.c b/wrapper.c
index ea3cf64d4c..1e45ab7b92 100644
--- a/wrapper.c
+++ b/wrapper.c
@@ -502,7 +502,7 @@ int g
compat-util.h is the first header (through xinclude.h)
Signed-off-by: Carlo Marcelo Arenas Belón
---
xdiff/xutils.c | 4
1 file changed, 4 deletions(-)
diff --git a/xdiff/xutils.c b/xdiff/xutils.c
index 963e1c58b9..cfa6e2220f 100644
--- a/xdiff/xutils.c
+++ b/xdiff/xutils.c
@@ -20,13 +20,9 @@
Signed-off-by: Carlo Marcelo Arenas Belón
---
Documentation/technical/api-trace2.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/technical/api-trace2.txt
b/Documentation/technical/api-trace2.txt
index 23c3cc7a37..f7ffe7d599 100644
--- a/Documentation/technical/
abort when hitting the first binary blob in a fixed match
after ed0479ce3d ("Merge branch 'ab/no-kwset' into next", 2019-07-15)
Signed-off-by: Carlo Marcelo Arenas Belón
---
grep.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/grep.c b/grep.c
index fc
94da9193a6 ("grep: add support for PCRE v2", 2017-06-01) allocate
a stack and assign it to a match context, but never pass it to
pcre2_jit_match, using instead the default.
Signed-off-by: Carlo Marcelo Arenas Belón
---
This might have positive performance consequences (per the com
abort when hitting the first binary blob in a fixed match
after ed0479ce3d ("Merge branch 'ab/no-kwset' into next", 2019-07-15)
Reviewed-by: Johannes Schindelin
Signed-off-by: Carlo Marcelo Arenas Belón
---
V2: spelling fixes from Eric Sunshine
grep.c | 4 ++--
1 file c
Sent as an RFC since it was meant to be applied against ab/pcre-jit-fixes
but that is likely to change with the reroll of that branch.
[PATCH 1/2] grep: make sure NO_LIBPCRE1_JIT disable JIT in PCRE1
[PATCH 2/2] grep: refactor and simplify PCRE1 support
The end result could be squashed together
pcre_study
with the right parameter after JIT support has been confirmed
and unless it was requested to be disabled with NO_LIBPCRE1_JIT
Signed-off-by: Carlo Marcelo Arenas Belón
---
grep.c | 15 +--
grep.h | 9 -
2 files changed, 9 insertions(+), 15 deletions(-)
diff --git a
d reliably to enforce
JIT doesn't get used.
Signed-off-by: Carlo Marcelo Arenas Belón
---
Makefile | 9 ++---
grep.h | 4 +---
2 files changed, 3 insertions(+), 10 deletions(-)
diff --git a/Makefile b/Makefile
index 11ccea4071..7e0e6cc129 100644
--- a/Makefile
+++ b/Makefile
@@ -
t unsigned char* vs const uint8_t*) to be shared.
Revert that change, as 94da9193a6 ("grep: add support for PCRE v2",
2017-06-01) failed to create an equivalent PCRE2 version.
Signed-off-by: Carlo Marcelo Arenas Belón
---
grep.c | 6 +++---
grep.h | 2 +-
2 files changed, 4 insertions(+
red by all functions.
Signed-off-by: Carlo Marcelo Arenas Belón
---
grep.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/grep.c b/grep.c
index cc65f7a987..d04635fad4 100644
--- a/grep.c
+++ b/grep.c
@@ -488,7 +488,6 @@ static void compile_pcre2_pattern(struct grep_pat *p, con
Just as it is done with PCRE1, make sure that the allocated chartables
get free at cleanup time.
This assumes no global context is used (NULL passed when created the
tables), but will likely be updated in tandem if that ever changes.
Signed-off-by: Carlo Marcelo Arenas Belón
---
grep.c | 1
codebase (ex: ab/pcre-jit-fixes) but hopefully the
spreading on short simple commits helps with reviewing.
Carlo Marcelo Arenas Belón (3):
grep: make pcre1_tables version agnostic
grep: use pcre_tables also for PCRE2
grep: plug leak of pcre chartables in PCRE2
grep.c | 12
From: Jeff King
We started including stdio.h to pick up the declaration of "FILE" in
f26a001226 (Enable wt-status output to a given FILE pointer.,
2007-09-17). But there's no need, since headers can assume that
git-compat-util.h has been included, which covers stdio.
This should just be redundan
From: Jeff King
There's no reason verify-tag.c needs to include signal.h. It's already
in git-compat-util.h, which we properly include as the first header.
And there doesn't seem to be a particular reason for this include; it's
just an artifact from the file creation in 2ae68fcb78 (Make verify-ta
8c912eea94 ("teach --histogram to diff", 2011-07-12) included them, but
were already part of xinclude.h
Signed-off-by: Carlo Marcelo Arenas Belón
---
xdiff/xhistogram.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/xdiff/xhistogram.c b/xdiff/xhistogram.c
index ec85f5992b..
92b7de93fb (Implement the patience diff algorithm, 2009-01-07) added them
but were already part of xinclude.h
Signed-off-by: Carlo Marcelo Arenas Belón
---
xdiff/xpatience.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/xdiff/xpatience.c b/xdiff/xpatience.c
index f3573d9f00..3c5601b602
stems (except for
the 3rd one) and that has since shown to also be needed
in Alpine Linux (because of _XOPEN_SOURCE redefinition).
The last 2 patches are new to the series and just cleanup
the dependency list in xdiff.
Carlo Marcelo Arenas Belón (3):
xdiff: drop system includes in xutils.c
e_tests.h:231:0: note: this is the location of the
previous definition
#define _FILE_OFFSET_BITS 32
Make sure git-compat-util.h is the first header (through xinclude.h)
Signed-off-by: Carlo Marcelo Arenas Belón
Signed-off-by: Junio C Hamano
---
V2: reword commit with feedback from Johannes
x
PCRE2 pattern 'foo bar', got '-48'
$ git grep -E 'foo bar'
fatal: Couldn't JIT the PCRE2 pattern 'foo bar', got '-48'
$ git grep -F 'foo bar'
fatal: Couldn't JIT the PCRE2 pattern 'foo bar', got '-4
when a JIT failure consistent with known security
restrictions is found at regex compilation time.
$ git grep 'foo bar'
fatal: Couldn't JIT the PCRE2 pattern 'foo bar', got '-48'
Signed-off-by: Carlo Marcelo Arenas Belón
---
V2: add command line to grep as suggest
leanup use free as there is no global context defined when
it was created (pcre2_maketables is passed a NULL pointer) but if that
gets ever changed will need to be updated in tandem.
Signed-off-by: Carlo Marcelo Arenas Belón
---
V2:
* better document why free is used as suggested by René
* avo
warning so that
corrective action could be taken.
[1] https://bugs.exim.org/show_bug.cgi?id=1749
Signed-off-by: Carlo Marcelo Arenas Belón
---
grep.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/grep.c b/grep.c
index f7c3a5803e..593a1cb7a0 100644
--- a/grep.c
+++ b/grep.c
(23.28+0.09) -1.5%
Signed-off-by: Carlo Marcelo Arenas Belón
---
Documentation/git-grep.txt | 2 +-
builtin/grep.c | 3 +--
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/Documentation/git-grep.txt b/Documentation/git-grep.txt
index 2d27969057..5d72e03b2e 100644
--- a
Add a baseline for a matching regex and make clear the distinction
between fixed (now using kwset) and a real simple expression.
Signed-off-by: Carlo Marcelo Arenas Belón
---
t/perf/p7810-grep.sh | 25 +++--
1 file changed, 23 insertions(+), 2 deletions(-)
diff --git a/t
mance
issue in HP-UX[1]
Lastly the performance numbers point to deficiencies in kwset and the
compat/regex code that will need to be addressed independently.
Carlo Marcelo Arenas Belón (2):
p7810: add more grep performance relevant cases
grep: make default number of threads reflect ru
https://public-inbox.org/git/pull.306.git.gitgitgad...@gmail.com
Signed-off-by: Carlo Marcelo Arenas Belón
---
Makefile | 2 +-
grep.c | 10 ++
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index bd246f2989..4b384f3759 100644
--- a/Makefile
there are still more things to consider as explained there
The third patch is the original leak patch rebased on top.
Carlo Marcelo Arenas Belón (3):
grep: make PCRE1 aware of custom allocator
grep: make PCRE2 aware of custom allocator
grep: avoid leak of chartables in PCRE2
Makefile |
Most of the code stolen from[1] to easy on comparison and including
the deficiency of setting the global context even for patterns that
won't need it.
Ideally, the call from grep_init could be moved to a place where it
could be set without needing a lock and at least with this approach
we have a p
-by: Carlo Marcelo Arenas Belón
---
grep.c | 7 ---
grep.h | 1 +
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/grep.c b/grep.c
index e748a6d68c..3e14ec91a6 100644
--- a/grep.c
+++ b/grep.c
@@ -524,7 +524,6 @@ static void compile_pcre2_pattern(struct grep_pat *p, const
struc
been tested and
considered mostly complete.
Junio, could you comment in my assumption that the use of grep in
revision.c doesn't require initializing a PCRE2 global context and
therefore not doing the cleanup?
Carlo Marcelo Arenas Belón (3):
grep: make PCRE1 aware of custom allocator
grep: make P
be good idea to test it in a platform where NED might have a
positive impact (ex: Windows 7)
[1] https://public-inbox.org/git/pull.306.git.gitgitgad...@gmail.com
Signed-off-by: Carlo Marcelo Arenas Belón
---
Makefile | 2 +-
grep.c | 10 ++
2 files changed, 11 insertions(+), 1 deletion(-
riginal code[1] this work was based on.
[1]
https://public-inbox.org/git/3397e6797f872aedd18c6d795f4976e1c579514b.1565005867.git.gitgitgad...@gmail.com/
Reported-by: Johannes Schindelin
Signed-off-by: Carlo Marcelo Arenas Belón
---
builtin/grep.c | 1 +
-by: Carlo Marcelo Arenas Belón
---
grep.c | 7 ---
grep.h | 1 +
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/grep.c b/grep.c
index 3e5bdf94a6..3d3ea0414e 100644
--- a/grep.c
+++ b/grep.c
@@ -527,7 +527,6 @@ static void compile_pcre2_pattern(struct grep_pat *p, const
struc
otherwise will warn on platforms where it is already defined (macOS)
convert the internal version from xdiff to a common one from
the compatibilty header additionally
this macro is commonly defined in system headers (usually )
but if it is not define it here so it can be used elsewhere
Signed-off-by: Carlo Marcelo Arenas Belón
---
git-compat-util.h | 5 +
sha256/block/sha256.c | 3 ---
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a
it is already defined whenever "sys/param.h" is available
Signed-off-by: Carlo Marcelo Arenas Belón
---
sha256/block/sha256.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/sha256/block/sha256.c b/sha256/block/sha256.c
index 18350c161a..0d4939cc2c 100644
--- a/sha256/block/sha2
Signed-off-by: Carlo Marcelo Arenas Belón
---
xdiff/xdiffi.c | 2 +-
xdiff/xemit.c | 6 +++---
xdiff/xhistogram.c | 6 +++---
xdiff/xmacros.h| 4 +---
xdiff/xprepare.c | 2 +-
5 files changed, 9 insertions(+), 11 deletions(-)
diff --git a/xdiff/xdiffi.c b/xdiff/xdiffi.c
index
contrib/nedmalloc doesn't support MALLOC_CHECK_ or MALLOC_PERTURB_
so add it to the same exception that is being used with valgrind
Signed-off-by: Carlo Marcelo Arenas Belón
---
t/test-lib.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/test-lib.sh b/t/test-lib.sh
after 36da893114 ("config.mak.dev: enable -Wunused-function", 2018-10-18)
macro generated code should use a similar solution than commit-slab to
silence the compiler.
Signed-off-by: Carlo Marcelo Arenas Belón
---
khash.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
di
in f48000fc ("Yank writing-back support from gitfakemmap.", 2005-10-08)
support for writting back changes was removed but the specific prot
flag that would be used was not checked for
Acked-by: Johannes Schindelin
Signed-off-by: Carlo Marcelo Arenas Belón
---
Changes in v2:
* re
after 36da893114 ("config.mak.dev: enable -Wunused-function", 2018-10-18)
it is expected to be used to prevent -Wunused-function warnings for code
that was macro generated
Signed-off-by: Carlo Marcelo Arenas Belón
---
commit-slab-impl.h | 4 ++--
git-compat-util.h | 2 ++
2 files
ands.c:23:1: warning: unused function 'kh_del_str' [-Wunused-function]
Reported-by: René Scharfe
Suggested-by: Nguyễn Thái Ngọc Duy
Signed-off-by: Carlo Marcelo Arenas Belón
---
khash.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/khash.h b/khash.h
index c0da40daa7..f
the macro generated code from delta-islands (using khash) triggers
some unused function warnings in macOS, OpenBSD and some linux with a
newer version of clang
Carlo Marcelo Arenas Belón (2):
commit-slabs: move MAYBE_UNUSED out
khash: silence -Wunused-function for delta-islands
commit-slab
sequencer.c: In function ‘write_basic_state’:
sequencer.c:2392:37: warning: zero-length gnu_printf format string
[-Wformat-zero-length]
write_file(rebase_path_verbose(), "");
Signed-off-by: Carlo Marcelo Arenas Belón
---
sequencer.c | 2 +-
1 file changed, 1 insertion(+),
nferred by the previous call
the somehow more convoluted syntax works around the issue by providing a non
empty format string and is already being used for the abort safety file since
1e41229d96 ("sequencer: make sequencer abort safer", 2016-12-07)
Signed-off-by: Carlo Marcelo Aren
after 36da893114 ("config.mak.dev: enable -Wunused-function", 2018-10-18)
it is expected to be used to prevent -Wunused-function warnings for code
that was macro generated
Signed-off-by: Carlo Marcelo Arenas Belón
Signed-off-by: Junio C Hamano
---
commit-slab-impl.h | 4 +---
git-com
message cleanup
* Make changes hdr-check clean
Changes from v1:
* Use MAYBE_UNUSED for all cases as suggested by Duy
Carlo Marcelo Arenas Belón (3):
commit-slab: move MAYBE_UNUSED into git-compat-util
khash: silence -Wunused-function in delta-islands from khash
commit-slab: missing definitions
ands.c:23:1: warning: unused function 'kh_del_str' [-Wunused-function]
Reported-by: René Scharfe
Suggested-by: Nguyễn Thái Ngọc Duy
Signed-off-by: Carlo Marcelo Arenas Belón
Signed-off-by: Junio C Hamano
---
khash.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/khas
struct commmit needs to be defined before commit-slab can generate
working code, object_id should be at least known through a forward
declaration
Signed-off-by: Carlo Marcelo Arenas Belón
---
commit-slab-impl.h | 2 ++
commit-slab.h | 2 ++
2 files changed, 4 insertions(+)
diff --git a
e_offset
Signed-off-by: Carlo Marcelo Arenas Belón
---
midx.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/midx.c b/midx.c
index 4fac0cd08a..a2c17e3108 100644
--- a/midx.c
+++ b/midx.c
@@ -710,7 +710,7 @@ static size_t write_midx_object_offsets(struct hashfile *f,
Signed-off-by: Carlo Marcelo Arenas Belón
---
Documentation/technical/api-parse-options.txt | 4
1 file changed, 4 deletions(-)
diff --git a/Documentation/technical/api-parse-options.txt
b/Documentation/technical/api-parse-options.txt
index 829b558110..2b036d7838 100644
--- a
511726e4b1 ("builtin/notes: fix premature failure when trying to add
the empty blob", 2014-11-09) removed the check for !len but left a
call to free the buffer that will be otherwise NULL
Signed-off-by: Carlo Marcelo Arenas Belón
---
builtin/notes.c | 4 +---
1 file changed, 1 inser
specially problematic in Windows where unsigned long is only 32bit wide
and therefore the assumption that a time_t would fit will lead to loss
of precision in a 64bit OS.
builtin/commit.c | 4 ++--
read-cache.c | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
unsigned long.
Signed-off-by: Carlo Marcelo Arenas Belón
---
builtin/commit.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/builtin/commit.c b/builtin/commit.c
index 0d9828e29e..a447e08f62 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -537,10 +537,10 @@ static i
oblematic so move to time_t instead.
Signed-off-by: Carlo Marcelo Arenas Belón
---
read-cache.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/read-cache.c b/read-cache.c
index 7b1354d759..5525d8e679 100644
--- a/read-cache.c
+++ b/read-cache.c
@@ -2625,9 +
ication times of a shared index.
dddbad728c ("timestamp_t: a new data type for timestamps", 2017-04-26)
shows why that might be problematic so move to timestamp_t/time_t.
if time_t can't represent a valid time keep the indexes for failsafe
Signed-off-by: Carlo Marcelo Arenas B
it://git.kernel.org/pub/scm/docs/man-pages/man-pages.git
warning: the following paths have collided (e.g. case-sensitive paths
on a case-insensitive filesystem) and only one from the same
colliding group is in the working tree:
'man2/_Exit.2'
'man2/_exit.2'
'man3/NAN
6c213e863a ("http-backend: respect CONTENT_LENGTH for receive-pack", 2018-07-27)
introduced all tests but without a check for CURL support from git.
Signed-off-by: Carlo Marcelo Arenas Belón
---
t/t5562-http-backend-content-length.sh | 6 ++
1 file changed, 6 insertions(+)
diff
tter to avoid any compiled
assumptions about the platform and let the user drive the fallback
through core.checkStat instead
Signed-off-by: Carlo Marcelo Arenas Belón
---
entry.c | 4
1 file changed, 4 deletions(-)
diff --git a/entry.c b/entry.c
index 0a3c451f5f..5ae74856e6 100644
--- a/
NetBSD added a BSD licensed reimplementation of GNU libintl to
its base at least since release 4.0 (mid 2012) and git can be
configured to build with it.
Signed-off-by: Carlo Marcelo Arenas Belón
---
INSTALL | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/INSTALL b
Likely still missing changes as it only completes a run with a minimal
number of dependencies but open for feedback
Requires pkgsrc packages for gmake, perl, bash and curl and completes a run
$ gmake SHELL_PATH=/usr/pkg/bin/bash NO_PYTHON=1 CURL_DIR=/usr/pkg test
Carlo Marcelo Arenas Belón (7
alternative fix
Signed-off-by: Carlo Marcelo Arenas Belón
---
t/t0301-credential-cache.sh | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/t/t0301-credential-cache.sh b/t/t0301-credential-cache.sh
index fd92533acf..9529c612af 100755
--- a/t/t0301-credential-cache.sh
+++ b/t
ional configure at least since NetBSD 6.0
Signed-off-by: Carlo Marcelo Arenas Belón
---
config.mak.uname | 1 +
1 file changed, 1 insertion(+)
diff --git a/config.mak.uname b/config.mak.uname
index 36c973c7e6..59ce03819b 100644
--- a/config.mak.uname
+++ b/config.mak.uname
@@ -246,6 +246,
had been doing as a workaround
Signed-off-by: Carlo Marcelo Arenas Belón
---
config.mak.uname | 1 +
1 file changed, 1 insertion(+)
diff --git a/config.mak.uname b/config.mak.uname
index 3ee7da0e23..36c973c7e6 100644
--- a/config.mak.uname
+++ b/config.mak.uname
@@ -253,6 +25
56ee96572a ("t5004: resurrect original empty tar archive test", 2013-05-09)
added a test to try to detect and workaround issues with the standard tar
from BSD, but at least in NetBSD would be better to instead require GNU tar
which is available from pkgsrc
Signed-off-by: Carlo Marc
ead so all tests from t5003 succeed
Signed-off-by: Carlo Marcelo Arenas Belón
---
t/test-lib.sh | 1 +
1 file changed, 1 insertion(+)
diff --git a/t/test-lib.sh b/t/test-lib.sh
index 6c6c0af7a1..2acb35f277 100644
--- a/t/test-lib.sh
+++ b/t/test-lib.sh
@@ -1244,6 +1244,7 @@ test_lazy_pre
otherwise will default to /usr/bin/perl which wouldn't normally exist
Signed-off-by: Carlo Marcelo Arenas Belón
---
config.mak.uname | 1 +
1 file changed, 1 insertion(+)
diff --git a/config.mak.uname b/config.mak.uname
index 59ce03819b..d2edb723f4 100644
--- a/config.mak.uname
that with the hope that in the future CI could be used for early detection
of similar issues
-Wpedantic is only enabled for clang 10 or higher (only available in macOS
with latest Xcode) but this restriction should be relaxed further as more
environments are tested
Signed-off-by: Carlo Marcelo Ar
b8cd1bb713 ("t6036, t6043: increase code coverage for file collision handling",
2018-11-07) uses this GNU extension that is not available in a POSIX complaint
cp; use cp -R instead
Signed-off-by: Carlo Marcelo Arenas Belón
---
to be applied on top of en/merge-path-collision for next
B3n?=
Subject: [PATCH] tests: add lint for non portable cp -a
cp -a, while a common flag isn't in POSIX and will therefore fail
on systems that don't have GNUish tools (like OpenBSD, AIX or Solaris)
Signed-off-by: Carlo Marcelo Arenas Belón
---
t/check-non-portable-shell.pl | 1 +
1 fi
anged the expected result for its tests from one containing an empty
directory to a plain empty archive but the portable test wasn't updated
resulting on them failing again in (at least) NetBSD and OpenBSD
Signed-off-by: Carlo Marcelo Arenas Belón
---
t/t5004-archive-corner-cases.sh | 17 ++
anged the expected result for its tests from one containing an empty
directory to a plain empty archive but the portable test wasn't updated
resulting on them failing again in (at least) NetBSD and OpenBSD
Signed-off-by: Carlo Marcelo Arenas Belón
---
t/t5004-archive-corner-cases.sh | 17 ++
6.4)
but considering this is a legacy feature it is likely that it affected
all old versions and is probably what most users had been using as a
workaround
Signed-off-by: Carlo Marcelo Arenas Belón
---
config.mak.uname | 1 +
1 file changed, 1 insertion(+)
diff --git a/config.mak.uname b/con
it is unconditionally initialized a few lines below
Signed-off-by: Carlo Marcelo Arenas Belón
---
unpack-trees.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/unpack-trees.c b/unpack-trees.c
index f25089b878..88dc9a615e 100644
--- a/unpack-trees.c
+++ b/unpack-trees.c
it is initialized unconditionally by a call to start_progress
below.
Signed-off-by: Carlo Marcelo Arenas Belón
---
midx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/midx.c b/midx.c
index ea2f3ffe2e..4fac0cd08a 100644
--- a/midx.c
+++ b/midx.c
@@ -941,7 +941,7 @@ static
NONCE_BAD is explicitly set when needed with the fallback
instead as NONCE_SLOP
Signed-off-by: Carlo Marcelo Arenas Belón
---
builtin/receive-pack.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/builtin/receive-pack.c b/builtin/receive-pack.c
index 95740f4f0e..ecce3d4043
introduced with c46c406ae1e (trace.h: support nested performance tracing)
on Aug 18, 2018 but not affecting maint
Signed-off-by: Carlo Marcelo Arenas Belón
---
read-cache.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/read-cache.c b/read-cache.c
index 1df5c16dbc
re keeping most
of the performance improvement.
Signed-off-by: Carlo Marcelo Arenas Belón
---
gettext.c | 14 +++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/gettext.c b/gettext.c
index d4021d690c..3ecf456f74 100644
--- a/gettext.c
+++ b/gettext.c
@@ -69,7 +69,14 @@
be good idea to test it in a platform where NED might have a
positive impact (ex: Windows 7)
[1] https://public-inbox.org/git/pull.306.git.gitgitgad...@gmail.com
Signed-off-by: Carlo Marcelo Arenas Belón
Signed-off-by: Junio C Hamano
---
Makefile | 2 +-
grep.c | 10 ++
2 files changed
-by: Carlo Marcelo Arenas Belón
Signed-off-by: Junio C Hamano
---
grep.c | 7 ---
grep.h | 1 +
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/grep.c b/grep.c
index 8ee982e2e8..ccb6ab38a3 100644
--- a/grep.c
+++ b/grep.c
@@ -541,7 +541,6 @@ static void compile_pcre2_patt
rest of the users of it added later.
Helped-by: René Scharfe
Reported-by: Johannes Schindelin
Signed-off-by: Carlo Marcelo Arenas Belón
---
V4:
* use xmalloc instead as suggested by René and Junio
* "fix" for regression in t7816 as reported by René
builtin/grep.c | 1 +
builtin/log.
)
or (hopefully not)
* ignore the original leak (maybe with an UNLEAK) as René suggested [3]
* discard this work and just use Dscho's fix (probably with some improvements)
Carlo Marcelo Arenas Belón (3):
grep: make PCRE1 aware of custom allocator
grep: make PCRE2 aware of custom alloc
@gmail.com/
[3] https://public-inbox.org/git/7ec60d57-9940-35f2-f7b5-c87d4dc7c...@web.de/
Carlo Marcelo Arenas Belón (3):
grep: make PCRE1 aware of custom allocator
grep: make PCRE2 aware of custom allocator
grep: avoid leak of chartables in PCRE2
Makefile | 2 +-
bu
t of the users of it added later.
Helped-by: René Scharfe
Reported-by: Johannes Schindelin
Signed-off-by: Carlo Marcelo Arenas Belón
---
builtin/grep.c | 1 +
grep.c | 56 +-
grep.h | 1 +
3 files changed, 57 insertions(+), 1 delet
be good idea to test it in a platform where NED might have a
positive impact (ex: Windows 7)
[1] https://public-inbox.org/git/pull.306.git.gitgitgad...@gmail.com
Signed-off-by: Carlo Marcelo Arenas Belón
Signed-off-by: Junio C Hamano
---
Makefile | 2 +-
grep.c | 10 ++
2 files changed
-by: Carlo Marcelo Arenas Belón
Signed-off-by: Junio C Hamano
---
grep.c | 7 ---
grep.h | 1 +
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/grep.c b/grep.c
index 8e0b838db0..146891e2bf 100644
--- a/grep.c
+++ b/grep.c
@@ -543,7 +543,6 @@ static void compile_pcre2_patt
Make using a general context (that is only needed with NED) to depend
on NED being selected at compile time.
the compile_context could be also make conditional but it gets ugly
really fasts with #ifdef
---
Makefile | 2 +-
grep.c | 4
2 files changed, 5 insertions(+), 1 deletion(-)
diff -
ismatch is unlikely to be found while testing because of that.
cURL is very strict about its allocator being thread safe and so that might
be an issue to look for.
Signed-off-by: Carlo Marcelo Arenas Belón
---
http.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/http.h b/http.h
index
USE_NED_ALLOCATOR=YesPlease is used (most
likely in Windows)
Signed-off-by: Carlo Marcelo Arenas Belón
---
This doesn't conflict with anything and was originally based on maint (so it
applies cleanly also to master and next), but is now rebased on top of
jk/drop-release-pack-memory so the final product wou
USE_NED_ALLOCATOR=YesPlease is used (most
likely in Windows)
Signed-off-by: Carlo Marcelo Arenas Belón
---
Notes:
v2: keep all global_init ifdefs together (as suggested by Junio)
http.h | 4
1 file changed, 4 insertions(+)
diff --git a/http.h b/http.h
index b429f1cf04..20a2030c94 100644
--- a/http.h
USE_NED_ALLOCATOR=YesPlease is used (most
likely in Windows)
Signed-off-by: Carlo Marcelo Arenas Belón
---
v3: proper use of #elif (Thanks Junio)
v2: keep all curl_global_init ifdefs together (as suggested by Junio)
http.h | 4
1 file changed, 4 insertions(+)
diff --git a/http.h b/http.h
index b429f1cf04
d reliably to enforce
JIT doesn't get used.
Signed-off-by: Carlo Marcelo Arenas Belón
---
Makefile | 9 ++---
grep.h | 4 +---
2 files changed, 3 insertions(+), 10 deletions(-)
diff --git a/Makefile b/Makefile
index f58bf14c7b..3f78ef942f 100644
--- a/Makefile
+++ b/Makefile
@@ -
pcre_study
with the right parameter after JIT support has been confirmed
and unless it was requested to be disabled with NO_LIBPCRE1_JIT
Signed-off-by: Carlo Marcelo Arenas Belón
---
grep.c | 16 ++--
grep.h | 9 -
2 files changed, 10 insertions(+), 15 deletions(-)
diff --git a
1 - 100 of 107 matches
Mail list logo