Re: Using "%define location_type" c++ skeleton should exclude default location

[Akim Demaille]

Le 14 avr. 2010 à 13:19, Akim Demaille a écrit :
> I have installed a first batch of changes in 2.5 and 2.6 to this end.  I 
> think I failed to keep you in CC, please have a look at the archive (it 
> should appear in 
> http://lists.gnu.org/archive/html/bison-patches/2010-04/threads.html I guess).

Installed in master and 2.5.

From af129079f04cc53129c9056af3ed3e3ae30fd10a Mon Sep 17 00:00:00 2001
From: Akim Demaille <address@hidden>
Date: Sat, 10 Apr 2010 13:38:37 +0200
Subject: [PATCH 03/18] tests: check fclose's return value.

        * tests/calc.at (_AT_DATA_CALC_Y): Check fclose's return status.
---
 ChangeLog     |    5 +++++
 tests/calc.at |    3 ++-
 2 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 5e38cd0..bc9cbc9 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2010-04-13  Akim Demaille  <address@hidden>
 
+       tests: check fclose's return value.
+       * tests/calc.at (_AT_DATA_CALC_Y): Check fclose's return status.
+
+2010-04-13  Akim Demaille  <address@hidden>
+
        tests: don't depend on the actual location type.
        * tests/calc.at: Use yy::parser::location_type rather than
        yy::location, since the former is always right, and might point to
diff --git a/tests/calc.at b/tests/calc.at
index 2f85baa..f102f95 100644
--- a/tests/calc.at
+++ b/tests/calc.at
@@ -346,7 +346,8 @@ main (int argc, const char **argv)
 ]AT_SKEL_CC_IF([], [m4_bmatch([$4], [%debug],
 [  yydebug = 1;])])[
   status = yyparse (]AT_PARAM_IF([[&result, &count]])[);
-  fclose (input);
+  if (fclose (input))
+    perror ("fclose");
   if (global_result != result)
     abort ();
   if (global_count != count)
-- 
1.7.0.5