On Tue, 29 May 2012, Christian Bruel wrote:

> I agree. I see two potential settings of the validated field. The %<
> that we just reviewed, and the case : /* We have Xno-YYY, search for
> XYYY.  */
> 
> It seems possible to remove those, I've just checked this during lunch
> :-) with the scenarios described above (without use m* f* specs).
> 
> If it is a prerequisite before my --spec patch, I'd like to propose it
> as an independent one, since this is a valid modification regardless of
> the --spec implementation (although it makes it clearer).

Please do send such a patch (with an explanation in each case of how 
"validated" still gets set with that redundant setting removed).

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to