Patch 978: Ignore an HLint suggestion in UI/Commands (and 1 more) - Darcs bug tracker

Title	Ignore an HLint suggestion in UI/Commands (and 1 more)
Superseder		Nosy List	owst
Related Issues
Status	accepted	Assigned To
Milestone

Created on 2012-11-11.17:09:50 by owst, last changed 2012-11-12.07:09:05 by ganesh.

Files
File name	Status	Uploaded	Type	Edit	Remove
ignore-an-hlint-suggestion-in-ui_commands.dpatch		owst, 2012-11-11.17:09:50	application/x-darcs-patch
patch-preview.txt		owst, 2012-11-11.17:09:50	text/x-darcs-patch
unnamed		owst, 2012-11-11.17:09:50

See mailing list archives for discussion on individual patches.

Messages
msg16339 (view)	Author: owst	Date: 2012-11-11.17:09:50
Tidying up UI/Commands 2 patches for repository http://darcs.net: Sun Nov 11 17:10:55 GMT 2012 Owen Stephens <darcs@owenstephens.co.uk> * Ignore an HLint suggestion in UI/Commands Sun Nov 11 17:11:16 GMT 2012 Owen Stephens <darcs@owenstephens.co.uk> * Tidy/Style UI/Commands Attachments ignore-an-hlint-suggestion-in-ui_commands.dpatch patch-preview.txt unnamed
msg16341 (view)	Author: owst	Date: 2012-11-11.17:11:07
Simple tidy-ups, self accepting, but depends on rebase
msg16370 (view)	Author: ganesh	Date: 2012-11-12.07:08:54
BTW this broke the Windows build by moving some imports inside a #ifdef. I guess HLint warns about two imports of the same module - we just need to be a bit careful where one of those imports is #ifdef'd

History
Date	User	Action	Args
2012-11-11 17:09:50	owst	create
2012-11-11 17:11:07	owst	set	status: needs-screening -> accepted-pending-tests messages: + msg16341
2012-11-12 01:35:59	owst	set	status: accepted-pending-tests -> accepted
2012-11-12 07:09:05	ganesh	set	messages: + msg16370