[Nouveau] [PATCH] drm/nouveau: remove redundant comments listing classes

Marcin Ślusarz marcin.slusarz at gmail.com
Tue Oct 16 22:44:23 PDT 2012


17 paź 2012 06:33, "Ben Skeggs" <skeggsb at gmail.com> napisał(a):
>
> On Tue, Oct 16, 2012 at 11:45:30PM +0200, Marcin Slusarz wrote:
> > Signed-off-by: Marcin Slusarz <marcin.slusarz at gmail.com>
> NACK on this one.  I did ponder doing such a thing myself when I first
> added the defines for the class names too, and decided against it as
> this will (at some point) become a user-facing header, and I was
> planning on documenting the differences between class revisions in the
> commented blocks.

I'm not against documentation, of course, but right now these comments are
useless. Can't they be added with real content later?

Marcin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20121017/f472f1fb/attachment.html>


More information about the Nouveau mailing list