Thanks Henry,

I have no further comments.

Good job!

Cheers,
Jim

Sheng-qi Jiang wrote:
> Jim Walker wrote:
>> Sheng-qi Jiang wrote:
>>>> usr/src/lib/ilmbase/libilmbase.3lib
>>
>> - "liblmbase(3LIB)" should be "libilmbase(3LIB)"
>>
>> This:
>> "ILM Base libraries were consisted by the following 4
>> libraries."
>> Should be:
>> "The Industrial Light & Magic (ILM) base package consists
>> of the following four libraries:"
>>
>> - "about the OpenEXR format" should be "about the OpenEXR
>> image file format." (note period (.) at end)
>> - remove extra blank lines between files in FILES section.
>> - remove extra blank lines before FILES, LICENSE and NOTES
>> sections.
>>
> fixed.
>>>> usr/src/lib/openexr/libopenexr.3lib
>>>>   33 cc [ \fIflag\fR\&.\&.\&.\ ] \fIfile\fR\&.\&.\&. 
>>>> -\fBI/usr/include/OpenEXR -\fBlIlmImf
>>>>   34 -\fBlImath -\fBlIex -\fBlHalf -\fBlIlmThread -\fBlz \fR [ 
>>>> \fIlibrary\fR\&.\&.\&. ]
>>>> - should only discuss IlmImf library now
>>> I would let it here to state the dependency.
>>
>> Ok.
>>
>> Change this:
>> "IlmImf is library to access the ILM  "OpenEXR"  file  format
>> for storing 16-bit FP images. Library in this package depend
>> on the IlmBase libraries."
>> To:
>> "The IlmImf library is used to read and write Industrial Light
>> & Magic (ILM) OpenEXR images. The IlmImf library depends on
>> the ilmbase libraries."
>>
>> - change "OpenEXR format" to "OpenEXR image file format".
>> - remove extra blank lines before FILES, SEE ALSO, LICENSE
>> and NOTES sections.
>>
>>
>> You need to add the attribute section like this to all man
>> pages:
>>
> Fixed.
>> ATTRIBUTES
>>      See attributes(5) for descriptions of the  following  attri-
>>      butes:
>>
>>      _______________________________________
>>     |   ATTRIBUTE TYPE   |  ATTRIBUTE VALUE|
>>     |_______________________________________
>>     | Availability       |  SUNWmeld       |
>>     |_______________________________________
>>     | Interface Stability|  Uncommitted    |
>>     |____________________|_________________|
>>
>> SEE ALSO
>>      diff(1), diff3(1)
>>
>> Cheers,
>> Jim
> Thanks.


Reply via email to