Hi Brian,
Attached are the updated version of the man page.
Best Regards,
-Alfred
Brian Cameron wrote:
>
> Alfred:
>
>> Thanks for the review. Attached is the updated man page for Songbird.
>
> Looks really good. Some minor comments:
>
> This manpage should include an OPERANDS section to describe the
> [URL] operand described in the SYNOPSIS
>
> songbird [options ... ] [URL]
>
> Note operands should not be in all-caps. It should be "url" and
> not "URL".
>
> > uniquely open approach to Internet digital media network
>
> I don't think Internet should be capitalized. Should just be
> "internet".
>
> > a wide range of audio formats(including mp3, AAC and ogg).
>
> Should have a space before the leading parens (the '(' character).
>
> I would not list mp3 or AAC in the manpage since these are not
> supported on Solaris in GStreamer by default. Instead might be
> better to just say "including raw audio, FLAC, Speex and ogg-vorbis"
>
> > gstreamer plugins to support some formats.
>
> Should be "GStreamer" and not "gstreamer". I'd say "media formats"
> instead of just "formats".
>
> > Songbird's add-ons system can be used to dynamically
> > personalize the media player.
>
> I think that should be "add-on system". Might be good to
> explain how you can dynamically personalize the player.
>
> > The Songbird Webpage API enables the ability to embed player
>
> "enables the ability to embed" is a little wordy. Would be good
> to more simply say "enables you to embed player"
>
> > functionality into the web site so it can interact with the
>
> I'd say "into a web site"
>
> The SEE ALSO section should refer to gst-std-options(5) and
> attributes(5)
>
> Brian
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: songbird.1.txt
URL:
<http://mail.opensolaris.org/pipermail/jds-review/attachments/20080819/e8d60432/attachment.txt>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://mail.opensolaris.org/pipermail/jds-review/attachments/20080819/e8d60432/attachment.html>
