Patch 978 Ignore an HLint suggestion in UI/Commands (and 1 more)

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

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

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.
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
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
Date User Action Args
2012-11-11 17:09:50owstcreate
2012-11-11 17:11:07owstsetstatus: needs-screening -> accepted-pending-tests
messages: + msg16341
2012-11-12 01:35:59owstsetstatus: accepted-pending-tests -> accepted
2012-11-12 07:09:05ganeshsetmessages: + msg16370