Alfred:
> Attached are the updated version of the man page.
Looks perfect to me. Some minor comments that you can fix below,
but you don't need to send this out for review again.
Songbird runs on Mozilla's XULRunner platform and utilizes
GStreamer media framework for media playback. It supports
I'd say "and utilizes the GStreamer media framework". Note
addition of word "the".
personalize the media player. Hundreds of useful add-ons
can be chosen to enhance Songbird.
Where do you get these add-ons?
-ProfileManager Start with profile manager.
Why would a user want to start with profile manager? Might
be nice to explain why this is useful.
gst-std-options(5), attributes(5)
The order should be numeric, then alphabetic so it shold
look like "bar(1), foo(1), bar(3), foo(3), bar(5), foo(5)".
So attributes(5) comes before gst-std-options(5).
For more information about songbird, please refer to
Shouldn't that be "Songbird"?
Brian
> 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
