Thread (14 messages) flat view 14 messages, 5 authors, 2016-06-15

Re: [PATCH] Optimize prefixcmp()

From: Johannes Schindelin <hidden>
Date: 2016-06-15 22:44:01

Hi,

On Sat, 29 Dec 2007, Marco Costalba wrote:
What your patch does not seem to avoid is a segfault if prefix or str 
are NULL pointers.
I am quite certain that it is not allowed to pass NULL pointers to strcmp, 
and even if it was, I maintain that it is bad style.

FWIW the test suite seems to agree with me, as it passes with my patch.

However, since you already seem to have a profiling setup ready, I would 
be interested in some numbers, i.e. if this patch is faster for you or 
slower, or shows no effect at all.

Ciao,
Dscho
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help