[llvm-commits] [PATCH] Coding Standards: Doxygen guidelines

Chandler Carruth chandlerc at google.com
Sat Oct 20 01:23:44 PDT 2012


Dmitri, all these patches look excellent. Please commit. We can tweak
this in post-commit review IMO.

On Fri, Sep 21, 2012 at 12:50 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> On Fri, Sep 21, 2012 at 10:42 PM, Sean Silva <silvas at purdue.edu> wrote:
>> Hi Dmitri, please make sure that you actually build the Sphinx docs
>> with the patch applied, as the current patch has a number of errors
>> which prevent it from rendering correctly. To do that, cd into docs/,
>> then run the command `make -f Makefile.sphinx html` and then open
>> docs/_build/html/CodingStandards.html in your web browser to make sure
>> that everything is being rendered correctly.
>
> Sorry for that.  I would certainly test my changes before committing,
> but it seems like there is a lot of discussion ahead, so I didn't
> bother to build.
>
> Dmitri
>
> --
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/



More information about the llvm-commits mailing list