On Sat, Mar 05, 2011 at 02:45:39PM +0100, [email protected] wrote:
> Author: kuemmel
> Date: Sat Mar  5 14:45:39 2011
> New Revision: 37859
> URL: http://www.lyx.org/trac/changeset/37859
> 
> Log:
> compile. TODO: review default parameter for replace_all
> 
> Modified:
>    lyx-devel/trunk/src/frontends/qt4/FindAndReplace.h
> 
> Modified: lyx-devel/trunk/src/frontends/qt4/FindAndReplace.h
> ==============================================================================
> --- lyx-devel/trunk/src/frontends/qt4/FindAndReplace.h        Sat Mar  5 
> 12:10:35 2011        (r37858)
> +++ lyx-devel/trunk/src/frontends/qt4/FindAndReplace.h        Sat Mar  5 
> 14:45:39 2011        (r37859)
> @@ -53,10 +53,14 @@
>       );
>  
>       /// Perform the scope-related buffer switch while searching
> -     void findAndReplaceScope(FindAndReplaceOptions & opt);
> +     bool findAndReplaceScope(FindAndReplaceOptions & opt, bool replace_all);
>  
>       /// Collect options from the GUI elements, then perform the search
> -     void findAndReplace(bool backwards, bool replace);
> +     bool findAndReplace(bool backwards, bool replace, bool replace_all = 
> false);
> +
> +     bool findAndReplace(bool casesensitive, bool matchword, bool backwards,
> +             bool expandmacros, bool ignoreformat, bool replace,
> +             bool keep_case, bool replace_all);

Shouldn't that be some kind of parameter stucture, instead of eight
booleans?

Andre'

Reply via email to