[MPlayer-dev-eng] [PATCH]VO_VDPAU, round 2

Benjamin Zores ben at geexbox.org
Tue Jan 6 20:25:32 CET 2009


Reimar Döffinger wrote:
> On Tue, Jan 06, 2009 at 09:31:07AM +0100, Benjamin Zores wrote:
>> I could only second that. Doxygen definitely is useless but for a library imho.
> 
> I forces at least a consistent style for documentation and it enforces
> a minimum amount of documentation.
> Obviously for functions that are part of the API documentation should
> not be duplicated, duplicated documentation is not that much better than
> duplicated code.

I'd agree on header files like libvo.h but doxygen on .c is pointless.


>> Efforts should be done on code, not on cosmetics, for once.
> 
> You assume that that saved effort would actually go into code.
> Also messed up indentation makes me loose interest in review very
> quickly, do you intend to review it and put effort into the code?

Sorry for that, I didn't meant indentation speaking about cosmetics but 
more about the comment stuff.

If it's to end up with something like that:

/*
  * \var1 Surely a useful variable
  * \var2 Probably another one
  * \brief Display picture
  * \return A status code
  */
int display_picture (int var1, int var2) {
   ...

   /* return success */
   return 0;
}

then I'm sorry but it's a waste of time.
It's better using meaningful variable and function names than
spaming code with useless doc/comments.

Comments are only there to explain why someone did something the way he 
did, not to describe the code.

Ben



More information about the MPlayer-dev-eng mailing list