Re: [boost] Re: Formal Review: The Spirit Parser Framework.

From: Joel de Guzman (djowel_at_[hidden])
Date: 2002-10-11 22:52:10

----- Original Message ----- 
From: "Carl Daniel" <cpdaniel_at_[hidden]>

> More review to come but first a round of typos and the like (there really
> are very few - I'm sure I've missed more than I've found, but overall, the
> documentation is very clean).
> 
> closures.html -
> 
> In the paragraph after "What are closures?", "exixting" should be "existing"
> 
> In the next paragraph, "What is more interesting is that the closure
> actually packages a local context (stack frame where some variables reside)
> and make them available outside the scope in which they actually exist."
> should read "What is more interesting is that the closure actually packages
> a local context (stack frame where some variables reside) and _makes_it_
> available outside the scope in which they actually exist." (underscores
> added to highlight the change).
> 
> In the example code under "free_closure", "my_closure3" should be
> "my_closure".
> 
> confix.html -
> 
> The very first sentence "Confix parsers are recognize" - remove "are".
> 
> later on, about 5 code snippets down "what in most cases is not"  - replace
> "what" with "which".
> 
> list_parsers.html -
> 
> "delimiter is an delimiter and end an optional closing expression" should be
> "delimiter is _a_ delimiter and end_is_ an optional closing expression".

Thanks! Done. Fixed in the CVS.
--Joel