On 12/14/20 7:19 AM, Ilya Leoshkevich wrote:
On Fri, 2020-12-11 at 07:51 -0500, Nathan Sidwell wrote:

I'm pretty sure this is wrong.  I think the test_text in
inclhack.def
should be a pre-fixed string that the testsuite presumably checks is
converted.

You're right; I've added your change from the Bugzilla and updated the
expectation.  Does the following look better?

Yes thanks. I don;t really understand why the base/sys/types.h change is needed, (surely there are other types.h fixincs?) But if this makes the tests pass, so be it!

nathan


After 92648faa1cb2 ("aix: Fixinclude") make check-fixincludes began to
fail (at least on gcc121 machine).  Fix by updating fixincludes/tests
and rerunning genfixes.

Co-developed-by: Nathan Sidwell <nat...@acm.org>

fixincludes/ChangeLog:

2020-12-11  Ilya Leoshkevich  <i...@linux.ibm.com>

        * fixincl.x: Rerun genfixes.
        * inclhack.def(aix_physadr_t): Change test_text to something
        that needs to be replaced.
        * tests/base/sys/types.h(aix_physadr_t): Add expectation.
---
  fixincludes/fixincl.x              | 4 ++--
  fixincludes/inclhack.def           | 2 +-
  fixincludes/tests/base/sys/types.h | 5 +++++
  3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/fixincludes/fixincl.x b/fixincludes/fixincl.x
index 21439652bce..cc17edfba0b 100644
--- a/fixincludes/fixincl.x
+++ b/fixincludes/fixincl.x
@@ -2,11 +2,11 @@
   *
   * DO NOT EDIT THIS FILE   (fixincl.x)
   *
- * It has been AutoGen-ed  October 21, 2020 at 10:43:22 AM by AutoGen 5.18.16
+ * It has been AutoGen-ed  December  9, 2020 at 11:16:08 AM by AutoGen 5.18.16
   * From the definitions    inclhack.def
   * and the template file   fixincl
   */
-/* DO NOT SVN-MERGE THIS FILE, EITHER Wed Oct 21 10:43:22 EDT 2020
+/* DO NOT SVN-MERGE THIS FILE, EITHER Wed Dec  9 11:16:08 EST 2020
   *
   * You must regenerate it.  Use the ./genfixes script.
   *
diff --git a/fixincludes/inclhack.def b/fixincludes/inclhack.def
index 80c9adfb07c..3a4cfe06542 100644
--- a/fixincludes/inclhack.def
+++ b/fixincludes/inclhack.def
@@ -731,7 +731,7 @@ fix = {
      select    = "typedef[ \t]*struct[ \t]*([{][^}]*[}][ \t]*\\*[ 
\t]*physadr_t;)";
      c_fix     = format;
      c_fix_arg = "typedef struct __physadr_s %1";
-    test_text = "typedef struct __physadr_s {";
+    test_text = "typedef  struct { random stuff } *       physadr_t;";
  };
/*
diff --git a/fixincludes/tests/base/sys/types.h 
b/fixincludes/tests/base/sys/types.h
index 683b5e93ecd..7340e76b175 100644
--- a/fixincludes/tests/base/sys/types.h
+++ b/fixincludes/tests/base/sys/types.h
@@ -9,6 +9,11 @@
+#if defined( AIX_PHYSADR_T_CHECK )
+typedef struct __physadr_s { random stuff } *  physadr_t;
+#endif  /* AIX_PHYSADR_T_CHECK */
+
+
  #if defined( GNU_TYPES_CHECK )
  #if !defined(_GCC_PTRDIFF_T)
  #define _GCC_PTRDIFF_T



--
Nathan Sidwell

Reply via email to