[boost] Re: Wave C++ Review Begins Today - February 7, 2005

From: Larry Evans (cppljevans_at_[hidden])
Date: 2005-02-11 21:54:37

On 02/11/2005 10:33 AM, Larry Evans wrote:
[snip]
> Should this rather be $(BOOST_ROOT)/libs/filesystem///boost_filesystem
> since I did find in $(BOOST_ROOT)/libs/filesystem/Jamfile.v2:
These should be:
{=======================================
Should this rather be
$(BOOST_ROOT)/libs/filesystem/build//boost_filesystem

since I did find in $(BOOST_ROOT)/libs/filesystem/build/Jamfile.v2:
}========================================
In addition, the zip file's .html files contain links to files in 
cvs.sourceforge.net instead of other files in the zip.  That would
be OK if the files were the same; however, in the zip file:

   libs/wave/doc/class_reference_context.html#header_synopsis

contains the boldface heading:

   Header wave/context.hpp synopsis

and, although not apparent in my mozilla browser, the characters:

   wave/context.hpp

are a link to:
 
http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/spirit/spirit/wave/wave/cpp_context.hpp?rev=HEAD&content-type=text/vnd.viewcvs-markup

which is confusing since the name is cpp_context.hpp instead of 
context.hpp as it appears in the heading.  Also, the sourceforge
cpp_context.hpp contains:

template <
     typename IteratorT,
     typename TokenT,
     typename InputPolicyT = 
iteration_context_policies::load_file_to_string,
     typename TraceT = trace_policies::default_tracing
 >
class context {

yet the zip file cpp_context.hpp contains:

  template <
     typename IteratorT,
     typename LexIteratorT,
     typename InputPolicyT = 
iteration_context_policies::load_file_to_string,
     typename TraceT = context_policies::default_preprocessing_hooks
 >
class context {

So which code should we be reviewing?