[FFmpeg-cvslog] r15741 - trunk/doc/general.texi
Thu Oct 30 11:01:10 CET 2008
On Wed, Oct 29, 2008 at 11:54:41PM +0100, Diego Biurrun wrote:
> On Wed, Oct 29, 2008 at 10:56:54AM +0100, Michael Niedermayer wrote:
> > On Wed, Oct 29, 2008 at 07:48:59AM +0100, diego wrote:
> > >
> > > Log:
> > > Add 'make checkheaders' to developer guidelines and patch checklist.
> > Now besides skiping others oppinions ...
> > This change is bad because
> > It adds one more point to a already rather long list, and one that is
> > really silly.
> I keep telling people that their patches need to pass 'make
> checkheaders' during reviews. Compn pointed out that this should really
> be listed in the policy/checklist. That was all.
Note that I have kept it in the patch submission checklist. Patch
submitters get to hear it often, so it might as well be in there.
More information about the ffmpeg-cvslog