Hi Thomas

Yes, that loop is superfluous. The default constructor for Point sets all the 
components to zero anyway,

best regards - Barry

On Friday 11 February 2011 13:33, Thomas Schoenemann wrote:
> Hi everybody!
>
> I hope this is the right place to ask this. If not, please correct me. I
> have a question on the file
>
> Optimizer.cpp
>
> in the directory
>
> mert
>
> (latest svn version).
>
> Specifically, this affects line 379 and the following ones:
>
>       Point direction;
>       for(unsigned int i=0;i<Point::getdim();i++)
>       direction[i]; // <<<<<<
>       direction[d]=1.0;
>
>
> The line marked with "<<<<<<" seems to be a statement without effect. I
> tested this on a vector of type float, and although I got no compiler
> warning the value at [i] remained unchanged. It seems likely to me that
> direction[i] was meant to be set to 0.0. Am I wrong?
>
> Best regards,
>   Thomas Schoenemann
> _______________________________________________
> Moses-support mailing list
> [email protected]
> http://mailman.mit.edu/mailman/listinfo/moses-support

-- 
The University of Edinburgh is a charitable body, registered in
Scotland, with registration number SC005336.

_______________________________________________
Moses-support mailing list
[email protected]
http://mailman.mit.edu/mailman/listinfo/moses-support

Reply via email to