Revert "Work-around bug in GCC 4.4.5" 20130509093229/pspp 20130510010502/pspp
authorJohn Darrington <john@darrington.wattle.id.au>
Thu, 9 May 2013 09:40:03 +0000 (11:40 +0200)
committerJohn Darrington <john@darrington.wattle.id.au>
Thu, 9 May 2013 09:40:03 +0000 (11:40 +0200)
This reverts commit c7711e39f523f206a74f5d0f52480ad10267e0ca.
The claim of a bug in GCC proved to be unfounded.  The problem which
this change tried to work-around has in fact been fixed in
commit 19bf192a093527bf10b1de95fbe8798ba156f960

src/data/casereader.c

index adb6b6a909b134e815836aa75cbf0b1263d79b68..c8050c6cde4c7b2a35d6d2b45aa752c3ecbee1e8 100644 (file)
@@ -242,19 +242,6 @@ casereader_count_cases__ (const struct casereader *reader,
   struct casereader *clone;
   casenumber n_cases;
 
-  /* This seems to avoid a bug in Gcc 4.4.5 where, upon
-     return from this function, the stack appeared corrupt,
-     and the program returned to the wrong address.  Oddly
-     the problem only manifested itself when used in conjunction
-     with the ODS reader, in code such as:
-
-     GET DATA /TYPE=ODS ....
-     LIST.
-  */
-#if (__GNUC__ == 4 ) && (__GNUC_MINOR__ == 4)
-  volatile int x = 1; x++;
-#endif
-
   clone = casereader_clone (reader);
   n_cases = casereader_advance (clone, max_cases);
   casereader_destroy (clone);