Coding style/Transition strategies - Revision history

TomYu: update and add reference to krb5-c-style.el

2009-11-03T05:02:45Z

update and add reference to krb5-c-style.el

TomYu: update for c-basic-offset vs c-file-style; add directory exception for libdb2

2009-07-14T13:23:17Z

update for c-basic-offset vs c-file-style; add directory exception for libdb2

TomYu: /* Implementation of transition */

2009-06-23T00:58:50Z

‎Implementation of transition

TomYu at 15:55, 22 June 2009

2009-06-22T15:55:08Z

TomYu at 13:08, 22 June 2009

2009-06-22T13:08:29Z

TomYu: New page: == Whitespace == To mark files as having all tabs expanded to space characters, place the following Emacs mode/local-variable line at the very top of the file:

 /* -*- mode: c; inde...

2008-12-17T21:02:14Z

New page: == Whitespace == To mark files as having all tabs expanded to space characters, place the following Emacs mode/local-variable line at the very top of the file: <pre> /* -*- mode: c; inde...

New page

== Whitespace ==

To mark files as having all tabs expanded to space characters, place the following Emacs mode/local-variable line at the very top of the file:

<pre>
/* -*- mode: c; indent-tabs-mode: nil -*- */
</pre>

@@ Line 40: / Line 40: @@
 </pre>
-We will devise some emacs customizations for automatically using the "krb5" c style where appropriate; putting a reference to that style in the local variable line would not be friendly to developers who do not define that style in their <code>.emacs</code> files, so we will use <code>c-basic-offset</code> instead.
+Some emacs customizations for automatically using the "krb5" c style are in {{trunkref|src/util/krb5-c-style.el}}; putting a reference to that style in the local variable line would not be friendly to developers who do not define that style in their <code>.emacs</code> files, so we will use the combination of <code>c-basic-offset</code> and <code>indent-tabs-mode</code> instead.
 For BSD-derived code, use
@@ Line 48: / Line 48: @@
 </pre>
-as this is already defined by most implementations of emacs CC Mode.
+as the "bsd" style is already defined by most implementations of emacs CC Mode.
 == Vim? ==

@@ Line 22: / Line 22: @@
 * src/appl/telnet
 * src/appl/gssftp
 == Whitespace ==
@@ Line 33: / Line 34: @@
 == Indentation ==
-To mark files as having indentation conforming to our coding style, add <code>c-file-style: "krb5"</code> to the local variable line:
+To mark files as having indentation conforming to our coding style, add <code>c-basic-offset: 4</code> to the local variable line:
 <pre>
-/* -*- mode: c; c-file-style: "krb5"; indent-tabs-mode: nil -*- */
+/* -*- mode: c; c-basic-offset: 4; indent-tabs-mode: nil -*- */
 </pre>
 For BSD-derived code, use
@@ Line 47: / Line 48: @@
 </pre>
 == Vim? ==

@@ Line 5: / Line 5: @@
 Multiple phases.  Consider batching changes.
 ** Mark with emacs file variables
 ** Confirm that this worked by checking if "<code>diff -w</code>" shows no changes

@@ Line 1: / Line 1: @@
 == Whitespace ==
@@ Line 27: / Line 51: @@
 == Vim? ==
-Do we want to add file modelines for vim?
+Do we want to add file modelines for vim?  Some versions of vim had vulnerabilities in handling of modelines.

← Older revision		Revision as of 13:08, 22 June 2009
Line 6:		Line 6:
	/* -- mode: c; indent-tabs-mode: nil -- */		/* -- mode: c; indent-tabs-mode: nil -- */
	</pre>		</pre>
		+
		+	== Indentation ==
		+
		+	To mark files as having indentation conforming to our coding style, add <code>c-file-style: "krb5"</code> to the local variable line:
		+
		+	<pre>
		+	/* -- mode: c; c-file-style: "krb5"; indent-tabs-mode: nil -- */
		+	</pre>
		+
		+	Remember to add the definition of the "krb5" c style to your <code>.emacs</code> file.
		+
		+	For BSD-derived code, use
		+
		+	<pre>
		+	/* -- mode: c; c-file-style: "bsd"; indent-tabs-mode: t -- */
		+	</pre>
		+
		+	If we get a style declaration for the "sun" C style, use that where appropriate as well (<code>src/lib/rpc</code> among others).
		+
		+	== Vim? ==
		+
		+	Do we want to add file modelines for vim?