[vlc-devel] [RFC 1/1] vlc_vector: add helpers for vectors

Rémi Denis-Courmont remi at remlab.net
Fri Aug 31 17:43:22 CEST 2018


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 
implementation.

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.

-- 
Реми Дёни-Курмон
http://www.remlab.net/





More information about the vlc-devel mailing list