[MFD] Apple has reviewed the System Object and System Control Service Semantics and has comments

[Michael Sweet]

Comments:

- Footer: Copyright needs to be "Copyright © 2011 The Printer Working Group. All rights reserved." (missing "The", extra comma, and some footers have 2010 listed)
- Header: Missing "Working Draft" prefix in title, which will be replaced by document number eventually...
- Lines 30-37: Centered but should be left justified
- Line 38: remove this line and extra page break
- Line 39: © instead of (C), no comma
- Boilerplate text before contents needs to be updated per current PWG document template.
- Line 104: "Table of Contents", not just "Contents"
- Fix table-of-contents indentation (probably a mismatch of the tab settings in the style)
- Line 155: "List of Figures"
- Line 167: "List of Tables"
- Line 177: Imaging Device duplicated.
- Line 178: Missing comma before "and the conformance requirements".
- General: I see "System object" (e.g. line 176) and "System Object" (e.g. line 181) used throughout; we should probably be consistent...
- Line 179: Here you use "Summary", but in other service specs you used "Overview"
- Line 235: remove
- Line 247: Stray period before 5108.01 reference
- Line 272: broken reference
- Line 281: Fix "DeviceS's"
- Section 6.3: You provide an example for the processing state. What about idle, etc? Perhaps provide a full list of states as a decision tree? e.g. If any service is in the "Processing" state, then the SystemStatus state is "Processing", otherwise if any service is in the "Stopped" state, then the SystemStatus state is "Stopped", etc.  
- Line 298: "The SystemControlService MODELS a set of elements ..."?
- Line 299: No comma before "and description".
- Lines 343-345: Not sure what happened here? Should be "Table 3 and Table 4 LIST the input and output ...", right?
- Lines 351, 354, 357: Table numbering looks wrong
- Lines 258-373: Numbers probably shouldn't be superscript here, but it looks like that is the convention used in previous (approved) documents...
- Line 381: Shouldn't DisableAllServices set the IsAccepting element to 'false'???
- Line 412: Page break for Table 5?
- Lines 437, 450: Put 'true' in single quotes as in other usage in the document?
- Line 506: broken reference
- Table 6: Maybe add whitespace to control wrapping in last column; hard to read as-is.
- Lines 553-554: Font size change?
- Section 12.1: Sort references
- Section 12.2: Remove or state there are none?


__________________________________________________
Michael Sweet, Senior Printing System Engineer, PWG Chair


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.pwg.org/pipermail/mfd/attachments/20111103/710ea295/attachment-0001.html>