Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(15)

Issue 1411123019: expand doxygen scope (Closed)

Created:
5 years, 1 month ago by rkotlerimgtec
Modified:
5 years, 1 month ago
CC:
native-client-reviews_googlegroups.com, rich.fuhler_imgtec.com
Base URL:
https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Make more visible with doxygen. To see the main effect of this, look at the doxygen for file IceGlobalContext.h to see the effect of turning this on. Without it, the nested classes are not available because they are private. My guess is that often doxygen is used to generate API documents and for that reason, exporting of private info is excluded by default. BUG= R=stichnot@chromium.org Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=9a10df7062c3bcb26a342f8e345acf16ecee7f7e

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M Doxyfile View 2 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
rkotlerimgtec
5 years, 1 month ago (2015-11-10 22:49:22 UTC) #3
rkotlerimgtec
On 2015/11/10 22:49:22, rkotlerimgtec wrote: If you look at the header for subzero doxygen, there ...
5 years, 1 month ago (2015-11-10 22:52:49 UTC) #4
Jim Stichnoth
lgtm
5 years, 1 month ago (2015-11-17 21:55:55 UTC) #5
Jim Stichnoth
5 years, 1 month ago (2015-11-18 14:25:52 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
9a10df7062c3bcb26a342f8e345acf16ecee7f7e (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698