* Cleanup committed
@ 1999-12-04 1:20 Lars Magne Ingebrigtsen
1999-12-04 18:07 ` Ben Rogers
1999-12-05 0:17 ` Dave Love
0 siblings, 2 replies; 6+ messages in thread
From: Lars Magne Ingebrigtsen @ 1999-12-04 1:20 UTC (permalink / raw)
I've just done a mop-up of "convention" fixes. That is, I've run
indent-sexp on everything, put double spaces after all full stops in
comments and doc strings, and made all doc strings have a complete
sentence as the first line.
This means that if you're planning on working on bits of the code, you
should check out the latest CVS before doing so. Otherwise, there'll
be lots of conflicts.
*phew* Two years to the next mop-up. :-) Or, if there was a
continual mop done before each release, then this wouldn't be an
issue... Hm. Perhaps I should run the indent-sexp command as part of
the release script? Yes, I think I'll do that.
--
(domestic pets only, the antidote for overdose, milk.)
larsi@gnus.org * Lars Magne Ingebrigtsen
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Cleanup committed
1999-12-04 1:20 Cleanup committed Lars Magne Ingebrigtsen
@ 1999-12-04 18:07 ` Ben Rogers
1999-12-05 0:17 ` Dave Love
1 sibling, 0 replies; 6+ messages in thread
From: Ben Rogers @ 1999-12-04 18:07 UTC (permalink / raw)
Lars Magne Ingebrigtsen <larsi@gnus.org> writes:
> *phew* Two years to the next mop-up. :-) Or, if there was a
> continual mop done before each release, then this wouldn't be an
> issue... Hm. Perhaps I should run the indent-sexp command as part of
> the release script? Yes, I think I'll do that.
I worked for a company which forced indentation checks as a part of
the CVS commit checking. While this is a little draconian it does
mean that there aren't any monstrous changes for re-indentation.
As long as you can run indent-sexp from the command line, it is pretty
simple to integrate into the CVS commit script.
ben
--
PGP Key fingerprint = 6A1B 3972 16F9 20E8 9F61 CCA3 AC38 7C55 EA00 89BE
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Cleanup committed
1999-12-04 1:20 Cleanup committed Lars Magne Ingebrigtsen
1999-12-04 18:07 ` Ben Rogers
@ 1999-12-05 0:17 ` Dave Love
1999-12-05 1:10 ` Lars Magne Ingebrigtsen
1 sibling, 1 reply; 6+ messages in thread
From: Dave Love @ 1999-12-05 0:17 UTC (permalink / raw)
In case it's not obvious, Checkdoc is useful in this context, though
not for dealing with indentation.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Cleanup committed
1999-12-05 0:17 ` Dave Love
@ 1999-12-05 1:10 ` Lars Magne Ingebrigtsen
1999-12-05 1:12 ` Lars Magne Ingebrigtsen
0 siblings, 1 reply; 6+ messages in thread
From: Lars Magne Ingebrigtsen @ 1999-12-05 1:10 UTC (permalink / raw)
Dave Love <d.love@dl.ac.uk> writes:
> In case it's not obvious, Checkdoc is useful in this context, though
> not for dealing with indentation.
Hm. Checkdoc complains about functions and variables missing doc
strings, which is not a style error, so it's not as usable as it
should be.
--
(domestic pets only, the antidote for overdose, milk.)
larsi@gnus.org * Lars Magne Ingebrigtsen
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~1999-12-05 1:17 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1999-12-04 1:20 Cleanup committed Lars Magne Ingebrigtsen
1999-12-04 18:07 ` Ben Rogers
1999-12-05 0:17 ` Dave Love
1999-12-05 1:10 ` Lars Magne Ingebrigtsen
1999-12-05 1:12 ` Lars Magne Ingebrigtsen
1999-12-05 1:17 ` Lars Magne Ingebrigtsen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).