[PATCH] tracedump: Add tri-state --color option (auto, always, or never)

[Carl Worth]

On Tue, 25 Oct 2011 21:53:35 -0400, Zack Rusin <zack at kde.org> wrote:
> That's almost there. We still do need to change the innamespace to 0 and 
> inline-open to 0, otherwise all of the inline funcs and all of our namespace 
> headers get wrong indention. 

Ah, I see that now. And I totally agree. It already drives me crazy that
with most C++ code I've seen the functions already aren't at the
left-most column due to an outer "class". Adding another level due to
"namespace" would be very annoying.

> does it but now emacs warns on startup that the file contains unsecure 
> variables. That's very inconvenient, I know there's some safe-local-variables 
> or such to deal with it, but I haven't had the time to figure out how to deal 
> with. If we can figure that out then we'll be good.

In response to the warning I get, I can hit '!' and emacs goes ahead and
edits my ~/.emacs for me, adding a safe-local-variables setting to the
custom-set-variables block. After that, the warning is gone.

So that's probably workable for a setting that we really do want.

-Carl

-- 
carl.d.worth at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/apitrace/attachments/20111025/9b5bb847/attachment.pgp>