[PATCH][Review request] AtomicExpr printed with extra comma

[Joe Sprowes]

Hi,

I am new to clang and want to get involved. Hopefully, someone will walk me through the process.

I found that atomic expressions were printed with an extra comma following the argument list:

For example:

__c11_atomic_init(&i, 0);

is printed as:

__c11_atomic_init(&i, 0, );

Attached is a patch to fix this.  I attached a test.

-Joe





> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130501/2003bfb9/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: StmtPrinter.patch
Type: application/octet-stream
Size: 1150 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130501/2003bfb9/attachment.obj>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130501/2003bfb9/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: AtomicExpr.cpp
Type: application/octet-stream
Size: 292 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130501/2003bfb9/attachment-0001.obj>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130501/2003bfb9/attachment-0002.html>