Review Request: Parser support for C99 designated initializers
Alexandre Courbot
gnurou at gmail.com
Sun Dec 18 12:51:53 UTC 2011
> On Dec. 18, 2011, 12:42 p.m., Steffen Ohrendorf wrote:
> > languages/cpp/parser/tests/test_parser.h, line 89
> > <http://git.reviewboard.kde.org/r/103448/diff/1/?file=43721#file43721line89>
> >
> > I'm not sure that C99 support should go into the C++ parser.
Where else? The C++ parser is currently rather a C/C++ one. KDevelop does not have separate support for C, so until this is available I do not see where this could go.
> On Dec. 18, 2011, 12:42 p.m., Steffen Ohrendorf wrote:
> > languages/cpp/parser/parser.cpp, line 2834
> > <http://git.reviewboard.kde.org/r/103448/diff/1/?file=43720#file43720line2834>
> >
> > try to avoid gotos
This makes error handling easier here. Not sure whether we need to rewind here or if a simple return would be enough - I have seen both through the code but could not figure out when each pattern should be used.
- Alexandre
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103448/#review9037
-----------------------------------------------------------
On Dec. 18, 2011, 12:23 p.m., Alexandre Courbot wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103448/
> -----------------------------------------------------------
>
> (Updated Dec. 18, 2011, 12:23 p.m.)
>
>
> Review request for KDevelop and Milian Wolff.
>
>
> Description
> -------
>
> Parser support for C99 designated initializers
>
> Support C99 initializers in the C++ parser, e.g:
>
> struct foo_t foo = {
> .has_cake = true,
> .nb_candles = 12,
> };
>
> int bar[10] = {
> [1] = 15,
> [9] = 25,
> };
>
>
> Diffs
> -----
>
> languages/cpp/parser/parser.h ffc3967e9bec09ff56204aab98e8f80ec6b036cf
> languages/cpp/parser/parser.cpp 1c9d9e403500f35761ebc6deb737a4a68a53c28d
> languages/cpp/parser/tests/test_parser.h fa92f1ce284df0936724f74f42f9ad6d4b3c97fc
> languages/cpp/parser/tests/test_parser.cpp f747cfa44bdf962be33cf97841fdae739b3e1771
>
> Diff: http://git.reviewboard.kde.org/r/103448/diff/diff
>
>
> Testing
> -------
>
> Used it for a couple of weeks, ensured the parser tests all pass.
>
>
> Thanks,
>
> Alexandre Courbot
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20111218/d0af6bf3/attachment.html>
More information about the KDevelop-devel
mailing list