[cfe-commits] [PATCH] -rewrite-includes

Lubos Lunak l.lunak at suse.cz
Fri Mar 16 14:34:35 PDT 2012


On Friday 16 of March 2012, Chandler Carruth wrote:
> On Thu, Mar 15, 2012 at 6:36 AM, Lubos Lunak <l.lunak at suse.cz> wrote:
> >  So the only remaing problem should be either finding a way of commenting
> > out
> > the directives properly or otherwise removing them.
>
> I've not looked at the patch in detail, but a simple idea: #if 0 block,
> with a comment explaining why.

 That's a very good idea and it seems to work nicely.

 I've also added commenting out #pragma once, and now the patch should be 
hopefully fully ready. I've also done a LibreOffice build again and this time 
there wasn't a single problem.

-- 
 Lubos Lunak
 l.lunak at suse.cz
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rewrite-includes.patch
Type: text/x-diff
Size: 21914 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120316/54eb16e6/attachment.patch>


More information about the cfe-commits mailing list