On Mon, 2 Oct 2006, Eric Y. Kow wrote:
I was thinking about it, and if nobody thinks it's too nitpicky, it would
be somewhat better if the string literal "--case-ok" wasn't used here
(and also in Add.lhs) I was tinkering with some code as shown below to
extract the long option name programmatically.
Good thought. Perhaps you could make a function like
getLongDarcsOption :: DarcsOption -> String
-- I am horrible with function names; pick something reasonable
So that you can just say (getLongDarcsOption allow_caseonly).
This has the added benefit of avoiding stuff like head.
Ok, I figured I'd make this:
DarcsArguments.getLongDarcsOptString :: DarcsOption -> String
Also, I see a lot of underscore-style functions like
get_long_darcs_opt_string. Is there a naming preference for this project?
Working on implementation, I needed to import to get this function for
DarcsArguments:
import DarcsCommands ( get_long_option )
And I'm getting an error:
Module imports form a cycle for modules:
Test
imports: IO DarcsUtils Lock RepoPrefs DarcsArguments DarcsRepo
PatchInfo PatchCore Patch Workaround Directory Monad
System DarcsUtils
DarcsArguments
imports: Bug DarcsCommands Workaround Printer SlurpDirectory
DarcsRepo Repository DarcsFlags PatchMatchData
FilePathUtils
DarcsURL RepoPrefs DarcsUtils PatchShow PatchInfo Patch
Char Monad
Maybe System List System.Directory System.Console.GetOpt
DarcsCommands
imports: Printer ArgumentDefaults DarcsUtils DarcsArguments
Workaround Global Test System.Exit Control.Exception
List Monad System.Console.GetOpt
ArgumentDefaults imports: RepoPrefs DarcsArguments Maybe Monad
make: *** [.depend] Error 1
Not sure what to do about it.
Thank you
--
.~. Dino Morelli
/V\ email: [EMAIL PROTECTED]
/( )\ irc: dwm
^^-^^ preferred distro: Debian GNU/Linux http://www.debian.org
_______________________________________________
darcs-devel mailing list
[email protected]
http://www.abridgegame.org/cgi-bin/mailman/listinfo/darcs-devel
