On Wed, Feb 08, 2006 at 11:12:40PM +0000, Mikhael Goikhman wrote:

> > > Thats a nice patch with an important feature.
> 
> I agree.
> 
> Can you please submit a patch after I commit the first pass on
> FvwmButtons indentation changes?

Ok, I'll wait and resubmit later.

> I also have a request to you. Your patch looks like:
> 
> +void ButtonPressProcess (b, act)
> +    button_info *b;
> +    char **act;
> +{
> +  FlocaleNameString tmp;
> +    int i,i2;
> +
> +        memset(&tmp, 0, sizeof(tmp));
> +     if (b && !*act && b->flags.b_Panel)
> +     {
> +       ActiveButton = b;
> 
> I.e. in one small piece of code you use four variations for indentation:
> 
>   * 2 spaces
>   * 4 spaces
>   * 8 spaces
>   * 1 tab

Well, actually I just copy-pasted this piece from function Loop() with minor
changes. Or maybe it was my misconfigured editor. Anyway I'll try to make code
cleaner.


-- 
Если таpакан замеp - вы философ.
George 'Nephrite' Potapov

Reply via email to