Bug 20034

Summary: [GTK] Invalid vim modeline in WebCore/plugins/gtk/gtk2xtbin.h (with patch)
Product: WebKit Reporter: Wouter Bolsterlee <uws+webkit>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Fix malformed Vim modeline eric: review+

Description Wouter Bolsterlee 2008-07-14 16:36:26 PDT
The vim modeline in WebCore/plugins/gtk/gtk2xtbin.h is malformed, causing an error when this file is opened in Vim.

(I'm atrevision 35167)
Comment 1 Wouter Bolsterlee 2008-07-14 16:40:15 PDT
Created attachment 22273 [details]
Fix malformed Vim modeline
Comment 2 Eric Seidel (no email) 2008-08-02 01:12:15 PDT
Comment on attachment 22273 [details]
Fix malformed Vim modeline

Looks sane enough.

Seems we should have a script to add WebKit-style compatible mode-lines to all files if we're going to use them.  It's silly to go through and fix each one by hand.
Comment 3 Jan Alonzo 2008-08-02 02:41:21 PDT
landed in r35526
Comment 4 Wouter Bolsterlee 2008-08-02 04:29:19 PDT
(In reply to comment #2)
> Seems we should have a script to add WebKit-style compatible mode-lines to all
> files if we're going to use them.  It's silly to go through and fix each one by
> hand.

Perhaps a sample Vim configuration file snippet can be placed somewhere in the WebKit tree. Some comments can be added to tell people how to use it, e.g. tell them to place something like this in their ~/.vimrc:

  autocmd BufReadPost /path/to/your/webkit/tree/* source /patch/to/your/webkit/tree/webkit.vim

Anyway, nothing major, so leaving this report RESOLVED FIXED.