[FFmpeg-cvslog] r15741 - trunk/doc/general.texi

[Diego Biurrun]

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.

Diego