Issue 5434108: Minor Doxyfile updates (Closed)
|
Created:
12 years, 5 months ago by epoger Modified:
12 years, 5 months ago CC:
skia-review_googlegroups.com Base URL:
http://skia.googlecode.com/svn/trunk/ Visibility:
Public. |
DescriptionMinor Doxyfile updates
BUG=404
Committed: http://code.google.com/p/skia/source/detail?r=2789
Patch Set 1 #Patch Set 2 : '' #MessagesTotal messages: 9
Ah, just saw your latest comment on the bug. I'll fix the list of commands so that filetypes allow for proper HTML display, and ping you when it's fixed.
Sign in to reply to this message.
LGTM. Nice!
Sign in to reply to this message.
On 2011/12/02 16:36:11, TomH wrote: > LGTM. Nice! Added commands to fix file mime-types. I'll go ahead and commit.
Sign in to reply to this message.
On 2011/12/02 17:13:27, epoger wrote: > On 2011/12/02 16:36:11, TomH wrote: > > LGTM. Nice! > > Added commands to fix file mime-types. I'll go ahead and commit. No GPU/PDF love? ;-)
Sign in to reply to this message.
On 2011/12/02 17:24:28, Steve VanDeBogart wrote: > On 2011/12/02 17:13:27, epoger wrote: > > On 2011/12/02 16:36:11, TomH wrote: > > > LGTM. Nice! > > > > Added commands to fix file mime-types. I'll go ahead and commit. > > No GPU/PDF love? ;-) Does doxygen generate PDFs?
Sign in to reply to this message.
On 2011/12/02 17:25:50, epoger wrote: > On 2011/12/02 17:24:28, Steve VanDeBogart wrote: > > On 2011/12/02 17:13:27, epoger wrote: > > > On 2011/12/02 16:36:11, TomH wrote: > > > > LGTM. Nice! > > > > > > Added commands to fix file mime-types. I'll go ahead and commit. > > > > No GPU/PDF love? ;-) > > Does doxygen generate PDFs? No. It looks like only core and effect is indexed.
Sign in to reply to this message.
No, the question is, what do we document internally? We had a discussion in CHAP yesterday and our conclusions were that we're committed to keeping core/ and effect/ public, but that we've talked fairly recently about pulling gpu/ and pdf/ out of the external API, so we didn't post them. The website will only contain documentation for the part of the API that clients are expected to access. Personally I want to have complete doxygen for the entire project at my fingertips. I'm pretty sure I can do that easily in a way that's maintainable enough to commit, and have a task to do so, but it's been pushed pretty far down my queue. Tom
Sign in to reply to this message.
My call. If we can produce an internal-dox and external-dox, then great, lets have the internal one include everything. However, I'm trying to limit caller's visibility into SkDevice and its subclasses (perhaps fruitlessly) for now, so I'd like the external-dox to be limited at the moment. On Fri, Dec 2, 2011 at 12:30 PM, <vandebo@chromium.org> wrote: > On 2011/12/02 17:25:50, epoger wrote: >> >> On 2011/12/02 17:24:28, Steve VanDeBogart wrote: >> > On 2011/12/02 17:13:27, epoger wrote: >> > > On 2011/12/02 16:36:11, TomH wrote: >> > > > LGTM. Nice! >> > > >> > > Added commands to fix file mime-types. I'll go ahead and commit. >> > >> > No GPU/PDF love? ;-) > > >> Does doxygen generate PDFs? > > > No. It looks like only core and effect is indexed. > > > http://codereview.appspot.com/5434108/ > > -- > You received this message because you are subscribed to the Google Groups > "skia-review" group. > To post to this group, send email to skia-review@googlegroups.com. > To unsubscribe from this group, send email to > skia-review+unsubscribe@googlegroups.com. > For more options, visit this group at > http://groups.google.com/group/skia-review?hl=en. >
Sign in to reply to this message.
I see. Fair enough -- most people only care about the public API.... However, that does technically include SkPDFDocument, but no biggie. -- Steve On Fri, Dec 2, 2011 at 9:32 AM, Mike Reed <reed@google.com> wrote: > My call. > > If we can produce an internal-dox and external-dox, then great, lets > have the internal one include everything. However, I'm trying to limit > caller's visibility into SkDevice and its subclasses (perhaps > fruitlessly) for now, so I'd like the external-dox to be limited at > the moment. > > On Fri, Dec 2, 2011 at 12:30 PM, <vandebo@chromium.org> wrote: > > On 2011/12/02 17:25:50, epoger wrote: > >> > >> On 2011/12/02 17:24:28, Steve VanDeBogart wrote: > >> > On 2011/12/02 17:13:27, epoger wrote: > >> > > On 2011/12/02 16:36:11, TomH wrote: > >> > > > LGTM. Nice! > >> > > > >> > > Added commands to fix file mime-types. I'll go ahead and commit. > >> > > >> > No GPU/PDF love? ;-) > > > > > >> Does doxygen generate PDFs? > > > > > > No. It looks like only core and effect is indexed. > > > > > > http://codereview.appspot.com/5434108/ > > > > -- > > You received this message because you are subscribed to the Google Groups > > "skia-review" group. > > To post to this group, send email to skia-review@googlegroups.com. > > To unsubscribe from this group, send email to > > skia-review+unsubscribe@googlegroups.com. > > For more options, visit this group at > > http://groups.google.com/group/skia-review?hl=en. > > > > -- > You received this message because you are subscribed to the Google Groups > "skia-review" group. > To post to this group, send email to skia-review@googlegroups.com. > To unsubscribe from this group, send email to > skia-review+unsubscribe@googlegroups.com. > For more options, visit this group at > http://groups.google.com/group/skia-review?hl=en. > >
Sign in to reply to this message.
|
