kleopatra999 / webm

Automatically exported from code.google.com/p/webm
0 stars 0 forks source link

wrong conventions? #848

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
1. What URL does this issue most closely relate to?
http://www.webmproject.org/code/contribute/conventions/

2. Please describe the issue in full.
According to the webpage, the astyle should be:
--style=bsd --min-conditional-indent=0 --break-blocks \
       --pad-oper --pad-header --unpad-paren \
       --align-pointer=name --indent-preprocessor \
       --indent-labels

BUT, in practice files in the libvpx do not follow this convention. A closer 
convention seems to be :
--style=google --indent=spaces=2 etc.

Is the website out of sync?

Original issue reported on code.google.com by galpin.f...@gmail.com on 2 Sep 2014 at 8:05

GoogleCodeExporter commented 9 years ago

Original comment by louquil...@google.com on 2 Sep 2014 at 5:53

GoogleCodeExporter commented 9 years ago
Yes, this is left-over for what vp8 was on release.
Moving forward it might be nice to bring both vp8/vp9 trees in line. I'm not 
sure what astyle does regarding tables, this has been one annoyance with 
clang-format. It might be worth another look after clang-3.5 is finalized.

Original comment by jz...@google.com on 3 Sep 2014 at 7:24

GoogleCodeExporter commented 9 years ago
Page update has propagated.

Original comment by louquil...@google.com on 24 Sep 2014 at 12:02