[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Emacs-diffs] scratch/merge-cedet-tests d08bd3b 117/316: C/C++: Fix func
From: |
Edward John Steere |
Subject: |
[Emacs-diffs] scratch/merge-cedet-tests d08bd3b 117/316: C/C++: Fix function pointer parsing |
Date: |
Sat, 28 Jan 2017 09:09:53 +0000 (UTC) |
branch: scratch/merge-cedet-tests
commit d08bd3b61ba26f2dcc95e892f54d9aa036830fa4
Author: David Engster <address@hidden>
Commit: Edward John Steere <address@hidden>
C/C++: Fix function pointer parsing
* semantic/bovine/c.by (variablearg-opt-name): Support parsing of
function pointers inside an argument list.
* semantic/bovine/c.el (semantic-c-reconstitute-token): Change
handling of function pointers; instead of seeing them as variables,
handle them as functions with a 'function-pointer' attribute. Also,
correctly deal with function pointers as function arguments.
(semantic-c-reconstitute-function-arglist): New function to parse
function pointers inside an argument list.
(semantic-format-tag-name): Use 'function-pointer' attribute instead
of the old 'functionpointer-flag'.
* tests/cedet/semantic/test/manual/cedet/testsppreplace.c:
* tests/cedet/semantic/test/manual/cedet/testsppreplaced.c: Disable
test which now breaks due to improved function pointer parsing.
This test is broken anyway, since it just succeeded because the
argument list was not parsed at all before this commit. The
underlying bug was always present in the semantic preprocessor. ---
tests/cedet/semantic/test/manual/cedet/testsppreplace.c | 8 ++++++++
1 file changed, 8 insertions(+)
---
test/manual/cedet/cedet/semantic/tests/testsppreplace.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/test/manual/cedet/cedet/semantic/tests/testsppreplace.c
b/test/manual/cedet/cedet/semantic/tests/testsppreplace.c
index 757a53c..ae04d07 100644
--- a/test/manual/cedet/cedet/semantic/tests/testsppreplace.c
+++ b/test/manual/cedet/cedet/semantic/tests/testsppreplace.c
@@ -151,11 +151,19 @@ OBJ(test) /* expands to ABtest */
/* TEST: Macro Recursion limits in arguments to a macro.
* This code is from ALSA (with names changed to moose), noticed by Yupeng. */
+
+/* David Engster:
+ This test is broken. The problem is not an infinite recursion, but that
+ the mr_moose macro shouldn't be applied when there are no arguments.
+ That this test succeeded was simply a bug in the function pointer
+ parser.
+
#define mr_moose(n) list_entry(n, struct mr_moose, list)
struct mr_moose_ops {
int (*mr_moose_disconnect)(struct mr_moose *dev);
};
+*/
/* END */
- [Emacs-diffs] scratch/merge-cedet-tests 96526d7 096/316: (Lisp): Add cit-uml.el (Misc): Add cit-uml.cgr, (continued)
- [Emacs-diffs] scratch/merge-cedet-tests 96526d7 096/316: (Lisp): Add cit-uml.el (Misc): Add cit-uml.cgr, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests 68ea5e3 098/316: (Lisp): Use preload compiler., Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests f300a36 100/316: (Lisp): Add cit-symref.el., Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests 007464d 104/316: (cit/Lisp): Add cit-projvar.el, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests a6f2191 103/316: (Lisp): Add cit-arduino, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests 302b4d7 107/316: eieio: Move to obsolete and use the one that ships with Emacs, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests ee352df 113/316: How to use the cit-test.sh script., Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests f91e2ce 114/316: New pretend system include for cpproot test., Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests d2dcc15 112/316: Move tests in cedet/semantic, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests 34041d4 120/316: Synchronize cedet/semantic with Emacs., Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests d08bd3b 117/316: C/C++: Fix function pointer parsing,
Edward John Steere <=
- [Emacs-diffs] scratch/merge-cedet-tests 3a7b458 110/316: Fixes for security tests, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests ceb9293 130/316: Move tests in cedet/semantic, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests 6d9366a 135/316: Make 'this' a pointer., Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests fb82965 136/316: Test `this' from in inline member functions, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests 64c305b 137/316: Test files contributed by Barry OReilly, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests c132878 122/316: Make unit tests use randomized file names, Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests 0897a28 127/316: Test file for Project.ede proj., Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests f3a937b 143/316: Test project file., Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests 876d3e9 138/316: New EDE project for testing dirmatch features., Edward John Steere, 2017/01/28
- [Emacs-diffs] scratch/merge-cedet-tests 8900419 148/316: Added arduino project detection, Edward John Steere, 2017/01/28