On 24.01.2009, at 22:29, Achim Schneider wrote:
Duncan Coutts <duncan.cou...@worc.ox.ac.uk> wrote:
930,931c930
< let lowerFirst (c:cs) = toLower c : cs
< in onlyPosIdent (posOf ide) (lowerFirst $ identToLexeme ide)
---
onlyPosIdent (posOf ide) (identToLexeme ide)
This one I don't understand. I don't understand why converting the
first char to lower case is bad here or how it relates to your
example.
Me neither. The problem is that, if the original is FunctionFoo and
the
renamed functionFoo, the original code thinks that they're the same
and
replaces functionFoo with Nothing, ...
Hi Achim,
I agree with Duncan that the second part of the patch
1086c1085
< adjustCase (c:cs) = toUpper c : map toLower cs
---
adjustCase (c:cs) = toUpper c : cs
is fine. I tried it, and it seems to work.
I think we would not want to apply the first part of the patch. The
problem is elsewhere, but I think I've already fixed this before.
Could you try whether the second part of the patch alone works fine
with the latest c2hs ?
If not, could you give a detailed description of the problem you're
experiencing when not applying the first part ?
benedikt
_______________________________________________
C2hs mailing list
C2hs@haskell.org
http://www.haskell.org/mailman/listinfo/c2hs
