[vlc-devel] [RFC 1/1] vlc_vector: add helpers for vectors
robux4 at ycbcr.xyz
Mon Sep 3 10:20:41 CEST 2018
On 31/08/2018 17:43, Rémi Denis-Courmont wrote:
> Le torstaina 30. elokuuta 2018, 17.29.14 EEST Romain Vimont a écrit :
>>> I think Rémi meant separate commit for implementation and test.
>> Ah ok! Sorry, I didn't get it.
>> However, I thought it was a good practice to include tests in the same
>> commit as the implementation.
> It is certainly commendable practice to include tests in the same push as
> I don't see the point putting them in the same commit, so long as incremental
> build is not broken. The smaller the commits, the better.
On the other hand if we revert the implementation commit, building the
tests is broken.
More information about the vlc-devel