|
From: | Daniel Stenberg |
Subject: | Re: [Bug-wget] [PATCH] Fix some clang-analyzer warnings |
Date: | Wed, 19 Nov 2014 09:59:43 +0100 (CET) |
User-agent: | Alpine 2.00 (DEB 1167 2008-08-23) |
On Wed, 19 Nov 2014, Darshit Shah wrote:
I already have a few sanity checks for Wget in my pre-commit hook. Such things can be restyled for a server side update hook too.
IMHO, it is better if you integrate those checks into the test suite or something so that contributors can figure them out and fix them already before sending off patches in the first place...
-- / daniel.haxx.se
[Prev in Thread] | Current Thread | [Next in Thread] |