clean up doxygen comments for main

clean up doxygen comments for main

the comments now for the main program describe it very well
and the classes and methods in the comments are clickable.

BUG=
R=stichnot@chromium.org

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=10665a23f5f2d5a59f225bd3e3073bd67d285d24

[rkotlerimgtec, 2015-12-08 23:48:22]

Description was changed from

==========
clean up doxygen comments for main

BUG=
==========

to

==========
clean up doxygen comments for main

the comments now for the main program describe it very well
and the classes and methods in the comments are clickable.

BUG=
==========

[rkotlerimgtec, 2015-12-08 23:48:22]

rkotlerimgtec@gmail.com changed reviewers:
+ jpp@chromium.org, kschimpf@chromium.org, stichnot@chromium.org

[rkotlerimgtec, 2015-12-08 23:49:24]

See the file for main in Doxygen after building with "docs".

On my machine it's in
file:///home/rkotler/nacl_dir/native_client/toolchain_build/src/subzero/docs/html/main_8cpp.html

[Jim Stichnoth, 2015-12-10 18:06:57]

https://codereview.chromium.org/1511483005/diff/1/src/main.cpp
File src/main.cpp (right):

https://codereview.chromium.org/1511483005/diff/1/src/main.cpp#newcode26
src/main.cpp:26: /// We can only compile the Ice::BrowserCompileServer object
with the NaCl
I think this is a bit unclear.  How about something like this:

We can only compile the Ice::BrowserCompileServer object with the PNaCl compiler
toolchain, when building Subzero as a sandboxed translator.

https://codereview.chromium.org/1511483005/diff/1/src/main.cpp#newcode28
src/main.cpp:28: ///
Remove this blank comment line?

[rkotlerimgtec, 2015-12-10 21:27:39]

https://codereview.chromium.org/1511483005/diff/1/src/main.cpp
File src/main.cpp (right):

https://codereview.chromium.org/1511483005/diff/1/src/main.cpp#newcode26
src/main.cpp:26: /// We can only compile the Ice::BrowserCompileServer object
with the NaCl
On 2015/12/10 18:06:56, stichnot wrote:
> I think this is a bit unclear.  How about something like this:
> 
> We can only compile the Ice::BrowserCompileServer object with the PNaCl
compiler
> toolchain, when building Subzero as a sandboxed translator.

Done.

https://codereview.chromium.org/1511483005/diff/1/src/main.cpp#newcode28
src/main.cpp:28: ///
On 2015/12/10 18:06:56, stichnot wrote:
> Remove this blank comment line?

Done.

[Jim Stichnoth, 2015-12-10 21:53:56]

lgtm

[Jim Stichnoth, 2015-12-10 21:57:35]

Description was changed from

==========
clean up doxygen comments for main

the comments now for the main program describe it very well
and the classes and methods in the comments are clickable.

BUG=
==========

to

==========
clean up doxygen comments for main

the comments now for the main program describe it very well
and the classes and methods in the comments are clickable.

BUG=
R=stichnot@chromium.org

Committed:
https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a...
==========

[Jim Stichnoth, 2015-12-10 21:57:36]

Committed patchset #2 (id:20001) manually as
10665a23f5f2d5a59f225bd3e3073bd67d285d24 (presubmit successful).