Patch 709: Readability tweaks on Darcs.Diff - Darcs bug tracker

Title	Readability tweaks on Darcs.Diff
Superseder		Nosy List	mndrix, wlangstroth
Related Issues
Status	accepted	Assigned To
Milestone

Created on 2012-01-29.05:01:56 by wlangstroth, last changed 2012-03-13.00:59:27 by gh.

Files
File name	Status	Uploaded	Type	Edit	Remove
patch-preview.txt		wlangstroth, 2012-01-29.05:01:55	text/x-darcs-patch
readability-tweaks-on-darcs_diff.dpatch		wlangstroth, 2012-01-29.05:01:55	application/x-darcs-patch
unnamed		wlangstroth, 2012-01-29.05:01:55

See mailing list archives for discussion on individual patches.

Messages
msg15070 (view)	Author: wlangstroth	Date: 2012-01-29.05:01:55
Adds a Haddock header, whitespace and standardized indentation to Darcs.Diff 1 patch for repository http://darcs.net: Sat Jan 28 23:58:17 EST 2012 Will Langstroth <will@langstroth.com> * Readability tweaks on Darcs.Diff Attachments patch-preview.txt readability-tweaks-on-darcs_diff.dpatch unnamed
msg15284 (view)	Author: mndrix	Date: 2012-03-12.22:32:14
As discussed in email, wlangstroth will remove the "------" separator lines in a separate patch. The style guide offers no guidance for organizing long import lines. The style used in this patch is certainly better than what was there before. I verified that all changes to treeDiff were whitespace-only cleanup with no effect on code behavior. Per PatchReview wiki, I "need a committer to push this patch"
msg15291 (view)	Author: gh	Date: 2012-03-13.00:59:26
Thanks for the review Michael, I'm pushing it.

History
Date	User	Action	Args
2012-01-29 05:01:56	wlangstroth	create
2012-02-22 21:14:45	ganesh	set	status: needs-screening -> needs-review
2012-03-12 22:32:15	mndrix	set	status: needs-review -> accepted-pending-tests nosy: + mndrix messages: + msg15284
2012-03-13 00:59:27	gh	set	status: accepted-pending-tests -> accepted messages: + msg15291