Apply suggestions from code review

Author: eemeli

spec/formatting.md

@@ -16,7 +16,8 @@ The formatting of a _message_ is defined by the following operations:
   For a message with no _selectors_, this is simple as there is only one _pattern_.
   With _selectors_, this will depend on their resolution.
 
-- **_<dfn>Formatting</dfn>_** takes the _resolved values_ of the selected _pattern_,
+- **_<dfn>Formatting</dfn>_** takes the _resolved values_ of
+  the _text_ and _placeholder_ parts of the selected _pattern_,
   and produces the formatted result for the _message_.
   Depending on the implementation, this result could be a single concatenated string,
   an array of objects, an attributed string, or some other locally appropriate data type.
@@ -272,7 +273,7 @@ the following steps are taken:
 
    The form that resolved _operand_ and _option_ values take is implementation-defined.
 
-   An implementation MAY pass additional arguments to the function,
+   An implementation MAY pass additional arguments to the _function handler_,
    as long as reasonable precautions are taken to keep the function interface
    simple and minimal, and avoid introducing potential security vulnerabilities.
 
@@ -286,16 +287,16 @@ the following steps are taken:
    additional detail about internal failures.
    Specifically, if the cause of the failure was that the datatype, value, or format of the
    _operand_ did not match that expected by the _function_,
-   the _function_ might cause a _Bad Operand_ error to be emitted.
+   the _function_ SHOULD cause a _Bad Operand_ error to be emitted.
   
    In all failure cases, use the _fallback value_ for the _expression_ as its _resolved value_.
 
 #### Function Handler
 
-To resolve a _function_,
-a **_<dfn>function handler</dfn>_** is required.
-This is an implementation-defined process such as a function or method
+A **_<dfn>function handler</dfn>_** is an implementation-defined process
+such as a function or method
 which accepts a set of arguments and returns a _resolved value_.
+A _function handler_ is required to resolve a _function_.
 
 An implementation MAY define its own functions and their handlers.
 An implementation MAY allow custom functions to be defined by users.

spec/registry.md

@@ -374,7 +374,7 @@ All other values produce a _Bad Operand_ error.
 ### Digit Size Options
 
 Some _options_ of number _functions_ are defined to take a "digit size option".
-_Function handlers_ of number _functions_ use these _options_ to control aspects of numeric display
+The _function handlers_ for number _functions_ use these _options_ to control aspects of numeric display
 such as the number of fraction, integer, or significant digits.
 
 A "digit size option" is an _option_ value that the _function_ interprets

spec/syntax.md

@@ -97,7 +97,7 @@ Attempting to parse a _message_ that is not _valid_ will result in a _Data Model
 
 A **_<dfn>message</dfn>_** is the complete template for a specific message formatting request.
 
-A **_<dfn>variable</dfn>_** is a _name_ associated with a _resolved value_.
+A **_<dfn>variable</dfn>_** is a _name_ associated to a _resolved value_.
 
 An **_<dfn>external variable</dfn>_** is a _variable_ 
 whose _name_ and initial value are supplied by the caller