X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fformats.c;h=32d638ae4a44f3fd0bed1ad7dd6484c77a4c52ea;hb=5c5204d433be6419112bd77f75b9a9adb642f766;hp=8e8e45af685354b121e132e47122e7ce7f672aad;hpb=06f9ee45954e5e71fa7f6262dbf37defa1dbf996;p=pspp

diff --git a/src/formats.c b/src/formats.c
index 8e8e45af68..32d638ae4a 100644
--- a/src/formats.c
+++ b/src/formats.c
@@ -14,8 +14,8 @@
 
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
-   Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
-   02111-1307, USA. */
+   Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+   02110-1301, USA. */
 
 #include <config.h>
 #include <limits.h>
@@ -28,6 +28,9 @@
 #include "str.h"
 #include "var.h"
 
+#include "gettext.h"
+#define _(msgid) gettext (msgid)
+
 #include "debug-print.h"
 
 enum
@@ -61,7 +64,7 @@ internal_cmd_formats (int which)
 {
   /* Variables. */
   struct variable **v;
-  int cv;
+  size_t cv;
 
   /* Format to set the variables to. */
   struct fmt_spec f;
@@ -70,14 +73,14 @@ internal_cmd_formats (int which)
   int type;
 
   /* Counter. */
-  int i;
+  size_t i;
 
   for (;;)
     {
       if (token == '.')
 	break;
 
-      if (!parse_variables (default_dict, &v, &cv, PV_SAME_TYPE))
+      if (!parse_variables (default_dict, &v, &cv, PV_NUMERIC))
 	return CMD_PART_SUCCESS_MAYBE;
       type = v[0]->type;
 
@@ -86,31 +89,11 @@ internal_cmd_formats (int which)
 	  msg (SE, _("`(' expected after variable list"));
 	  goto fail;
 	}
-      if (!parse_format_specifier (&f, 0) || !check_output_specifier (&f))
+      if (!parse_format_specifier (&f, 0)
+          || !check_output_specifier (&f, true)
+          || !check_specifier_type (&f, NUMERIC, true))
 	goto fail;
 
-      /* Catch type mismatch errors. */
-      if ((type == ALPHA) ^ (0 != (formats[f.type].cat & FCAT_STRING)))
-	{
-	  msg (SE, _("Format %s may not be assigned to a %s variable."),
-	       fmt_to_string (&f), type == NUMERIC ? _("numeric") : _("string"));
-	  goto fail;
-	}
-
-      /* This is an additional check for string variables.  We can't
-         let the user specify an A8 format for a string variable with
-         width 4. */
-      if (type == ALPHA)
-	{
-	  /* Shortest string so far. */
-	  int min_len = INT_MAX;
-
-	  for (i = 0; i < cv; i++)
-	    min_len = min (min_len, v[i]->width);
-	  if (!check_string_specifier (&f, min_len))
-	    goto fail;
-	}
-
       if (!lex_match (')'))
 	{
 	  msg (SE, _("`)' expected after output format."));