[pacman-dev] [PATCH] Remove ts and sw from vim modeline when noet is set

[Allan McRae]

On 27/01/14 06:45, Dan McGee wrote:
> On Wed, Jan 22, 2014 at 5:06 PM, Florian Pritz <bluewind at xinu.at> wrote:
> 
>> Forcing vim users to view files with a tabstop of 2 seems really
>> unnecessary when noet is set. I find it much easier to read code with
>> ts=4 and I dislike having to override the modeline by hand.
>>
>> Command run:
>> find . -type f -exec sed -i '/vim.* noet/s# ts=2 sw=2##' {} +
>>
>> Signed-off-by: Florian Pritz <bluewind at xinu.at>
>> ---
>>
> I'm fine with this. Might be wise to add a note in HACKING or somewhere
> that the preferred 80 character width might only hold true if tabs are
> viewed as two spaces wide.
> 

Added:

diff --git a/HACKING b/HACKING
index cc4983f..03605dd 100644
--- a/HACKING
+++ b/HACKING
@@ -10,7 +10,9 @@ Coding Style
 ------------

 1.  All code should be indented with tabs. (Ignore the use of only
spaces in
-    this file) By default, source files contain the following Vim modeline:
+    this file.) A tab size of two spaces is used when calculating line
widths,
+    which should be a maximum of 80 characters. By default, source files
+    contain the following Vim modeline:
 +
 [source,C]
 -------------------------------------------