Thread (13 messages) flat view 13 messages, 4 authors, 2016-12-22

Re: [PATCH] mailinfo.c: move side-effects outside of assert

From: Johannes Schindelin <hidden>
Date: 2016-12-20 14:13:00

Hi Kyle,

On Mon, 19 Dec 2016, Kyle J. McKay wrote:
On Dec 19, 2016, at 09:45, Johannes Schindelin wrote:
quoted
ACK. I noticed this problem (and fixed it independently as a part of a
huge patch series I did not get around to submit yet) while trying to
get Git to build correctly with Visual C.
Does this mean that Dscho and I are the only ones who add -DNDEBUG for
release builds?  Or are we just the only ones who actually run the test
suite on such builds?
It seems you and I are for the moment the only ones bothering with running
the test suite on release builds.

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