OASIS Mailing List ArchivesView the OASIS mailing list archive below
or browse/search using MarkMail.

 


Help: OASIS Mailing Lists Help | MarkMail Help

obix message

[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]


Subject: [OASIS Issue Tracker] Commented: (OBIX-85) Various editorial issues


    [ http://tools.oasis-open.org/issues/browse/OBIX-85?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=36170#action_36170 ] 

Matthias Hub commented on OBIX-85:
----------------------------------

Thanks for the thorough review!

Line 137: "together the" needs a comma or a "then"
> done

Line 139: a comma is needed after "context"
> done

Line 158: "MUST not be empty" is redundant if you are saying that it must contain other objects
> done

Line 167: Gramatically incorrect - "exchange pattern the REST-style"
> I changed that to: "As WebSocket follows a message exchange pattern the REST-style messages of OBIX needs to be wrapped."

Line 168: Missing comma after "For that"
> done

Line 169: Missing comma after "After a successful connection to the OBIX server"
> done

Line 170: "make operation"...make is a proper term and should be distinguished as such with bold or italicized font
Line 171: "behaves the same than"...the "than" is inappropriate
Line 201: The term "contract" should be capitalized
Line 230: missing a comma after "is open"
Line 231: " a property of the watched objects is changed"...should be ...that is changed
> removed as it is now in core spec

Line 271: "must" should be in all caps
> done

> Various editorial issues
> ------------------------
>
>                 Key: OBIX-85
>                 URL: http://tools.oasis-open.org/issues/browse/OBIX-85
>             Project: OASIS Open Building Information Exchange (oBIX) TC
>          Issue Type: Bug
>          Components: WebSockets
>    Affects Versions: WS WD09
>            Reporter: Chris Bogen
>            Assignee: Matthias Hub
>            Priority: Minor
>
> There are various grammatical and stylistic issues that should be addressed before public review.  
> Line 137:  "together the" needs a comma or a "then"
> Line 139:  a comma is needed after "context"
> Line 158:  "MUST not be empty" is redundant if you are saying that it must contain other objects
> Line 167:  Gramatically incorrect - "exchange pattern the REST-style"
> Line 168:  Missing comma after "For that"
> Line 169:  Missing comma after "After a successful connection to the OBIX server"
> Line 170:  "make operation"...make is a proper term and should be distinguished as such with bold or italicized font
> Line 171:  "behaves the same than"...the "than" is inappropriate
> Line 201:  The term "contract" should be capitalized 
> Line 230:  missing a comma after "is open"
> Line 231: " a property of the watched objects is changed"...should be ...that is changed
> Line 271:  "must" should be in all caps

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://tools.oasis-open.org/issues/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]