Opened 11 years ago

#43 new defect

Small fixes

Reported by: worley@… Owned by:
Priority: minor Milestone:
Component: rfc4244bis Version:
Severity: In WG Last Call Keywords:
Cc:

Description

This ticket is a collection of items that are either editorial nits
(things that an editor might correct) or "minor" wording improvements
(things that require content knowledge to fix but seem to have no
interesting/controversial technical content). Several of these have
been consolidated from previous tickets (as noted).

--- Ticket #30

Stray double-quote:

History-Info is a header field as defined by [RFC3261]. "It may

Missing space:

is set to a value of "mp".The "mp" attribute in the hi-target

--- Ticket #33

I see the sentence "The following summarizes the syntax..." -- this
suggests that the actual syntax definition/specification is
elsewhere. Do you mean "The following is the syntax..."?

--- Ticket #31

In section 6.1 is:

The following rules apply:

o There MUST be exactly one hi-index parameter per hi-entry.

o There MUST be no more than one hi-target parameter.

o There MAY be any number of hi-extension parameters.

o The ABNF definitions for "generic-param" and "name-addr" are
from [RFC3261].

The fourth item is different from the first three and should be put
elsewhere.

The second and third items also need "... per hi-entry". But since
that would apply that qualifier to all 3 of the remaining items in the
list, it would be better to phrase it:

The following rules apply to each hi-entry:

o There MUST be exactly one hi-index parameter.

o There MUST be no more than one hi-target parameter.

o There MAY be any number of hi-extension parameters.

--- Ticket #36

Section 6.3.1 para 1 starts:

This specification defines a new SIP/SIPS URI Parameter indicating
the mechanism by which a new target for a request is determined.

However the actual name of the parameter is not given until para 5. I
suggest putting the name of the parameter into the first sentence of
para 1.

--- Ticket #39

Section 6.3.1 para 5 is:

The following defines the ABNF for the "hit" URI parameter:

Please change this to:

The ABNF for the "hit" URI parameter is:

(The object of "define" is always a word or term that is defined; it
is not the ABNF that is defined. Indeed, the ABNF *defines* the
syntax of the "hit" URI parameter.)

--- Ticket #17

In section 3 is the odd phrase "any valid response" -- are there
invalid responses? What is validity of responses? Better would be "any
response", or probably more informatively, "any provisional or final
response".

--- Ticket #26

In section 5.2 is "SHOULD forward captured History-Info in subsequent,
provisional, and final responses to the request". I think that
"subsequent" adds no meaning and should be removed.

--- Ticket #22

This:

A UAC that does not want an hi-entry added due to privacy
considerations MUST include a Privacy header with a priv-value(s)
of "header" or "history." A UAC that wants to ensure that privacy
not be applied to its identity MUST include a Privacy header with
a priv- value of "none."

should have the punctuation moved for clarity:

A UAC that does not want an hi-entry added due to privacy
considerations MUST include a Privacy header with a priv-value(s)
of "header" or "history". A UAC that wants to ensure that privacy
not be applied to its identity MUST include a Privacy header with
a priv-value of "none".

Change History (0)

Note: See TracTickets for help on using tickets.