Re: [PATCH] gui: Fix sense of "Flag Significant Correlations" checkbox.

[Ben Pfaff]

How odd.

I'll drop this patch, then.

Thank you for the review.

John Darrington <address@hidden> writes:

> According to the SPSS (v17) manual "SIG" means do NOT flag
> significant values, and "NOSIG" means DO flag them.  
>
> On Sun, Aug 08, 2010 at 03:07:06PM -0700, Ben Pfaff wrote:
>      This was doing the opposite of what it should have.
>      
>      Reported by Harry Thijssen <address@hidden>.
>      ---
>       src/ui/gui/correlation-dialog.c |    6 +++---
>       1 files changed, 3 insertions(+), 3 deletions(-)
>      
>      diff --git a/src/ui/gui/correlation-dialog.c 
> b/src/ui/gui/correlation-dialog.c
>      index afc2344..84157cf 100644
>      --- a/src/ui/gui/correlation-dialog.c
>      +++ b/src/ui/gui/correlation-dialog.c
>      @@ -1,5 +1,5 @@
>       /* PSPPIRE - a graphical user interface for PSPP.
>      -   Copyright (C) 2009  Free Software Foundation
>      +   Copyright (C) 2009, 2010  Free Software Foundation
>       
>          This program is free software: you can redistribute it and/or modify
>          it under the terms of the GNU General Public License as published by
>      @@ -164,9 +164,9 @@ generate_syntax (const struct correlation *rd)
>       
>       
>         if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON 
> (rd->significant)))
>      -    g_string_append (string, " NOSIG");
>      -  else
>           g_string_append (string, " SIG");
>      +  else
>      +    g_string_append (string, " NOSIG");
>       
>       
>         g_string_append (string, ".\n");
>      -- 
>      1.7.1
>      
>      
>      _______________________________________________
>      pspp-dev mailing list
>      address@hidden
>      http://lists.gnu.org/mailman/listinfo/pspp-dev

-- 
Ben Pfaff 
http://benpfaff.org