Review Request: Let __builtin_offsetof be used with "struct foo" as type

[Alexandre Courbot]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104912/
-----------------------------------------------------------

(Updated May 14, 2012, 1:34 a.m.)


Review request for KDevelop.


Changes
-------

Added unit tests for both struct and typedef'd struct.


Description
-------

There is a clever macro defined that supports __builtin_offset, but constructs of the form "__builtin_offset(struct foo, member)" would fail with a parse error. This patch adds parentheses around the type to ensure these are supported as well.


Diffs (updated)
-----

  languages/cpp/setuphelpers.cpp 23d549e 
  languages/cpp/tests/test_cppcodecompletion.cpp 67a3033 

Diff: http://git.reviewboard.kde.org/r/104912/diff/


Testing
-------

Checked it worked, the change is pretty simple.


Thanks,

Alexandre Courbot

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120514/2e5bc20c/attachment.html>