Re: [CEDET-devel] Parsing standard C++ includes

[DaveS <da...@te...>]

On Sun, 22 Feb 2009 10:11:02 +1100 Alastair Rankine <ar...@in...> wrote:
> On 21/02/2009, at 11:26 PM, Eric M. Ludlam wrote:
>

[...]

>
> Yep, well it's setting the mode using set-auto-mode, but this seems to  
> be relying on local variables, which is disabled in semantic-find-file- 
> noselect. Leaving local variables enabled seems to fix the problem:
>
> === modified file 'semantic/semantic-fw.el'
> --- semantic/semantic-fw.el	2009-02-04 12:45:50 +0000
> +++ semantic/semantic-fw.el	2009-02-21 22:57:29 +0000
> @@ -430,7 +430,7 @@
>   	 ;; Don't prompt to insert a template if we visit an empty file
>   	 (auto-insert nil)
>   	 ;; We don't want emacs to query about unsafe local variables
> -	 (enable-local-variables nil)
> +	 ;(enable-local-variables nil)
>   	 ;; ... or eval variables
>   	 (enable-local-eval nil)
>   	 )
>
> This isn't an ideal fix; as the comment indicates there is still the  
> possibility of a warning about unsafe local variables. However I don't  
> see an alternative - according to the doco for enable-local-variables,  
> it also controls setting the mode appropriately.

I'm the one that requested the change for enable-local-variables (the
warnings about unsafe local variables can really get annoying).  The docs
are a bit unclear but I wonder if setting enable-local-variables to :safe
will be enough for the set-auto-mode magic to still work.  If so then I
think everyone wins.

-- 
DS