Last modified: 2014-11-02 14:03:16 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T54509, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52509 - Haskell: highlights single quote as strings
Haskell: highlights single quote as strings
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
SyntaxHighlight (GeSHi) (Other open bugs)
master
All All
: Low minor with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-08-04 00:19 UTC by feiyuanbw
Modified: 2014-11-02 14:03 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description feiyuanbw 2013-08-04 00:19:51 UTC
Bug: In the lang="haskell" mode, any single quotes (apostrophes) are treated as if they were delimiters for a string.

Expected: Single quotes should be treated as normal characters used for identifiers.
Comment 1 Daniel Mlot 2013-09-28 21:30:24 UTC
The effects of this issue in the Haskell Wikibook are nasty [1]. Upstream, here is a bug report [2] and a proposed patch [3], but both were seemingly ignored.

[1] For instance, https://en.wikibooks.org/w/index.php?title=Haskell/Understanding_monads/State&oldid=2550766#Implementation_with_Functional_Purity .

[2] http://sourceforge.net/p/geshi/bugs/217/

[3] http://sourceforge.net/p/geshi/bugs/219/
Comment 2 Gerrit Notification Bot 2013-09-28 22:29:39 UTC
Change 86395 had a related patch set uploaded by Ori.livneh:
Haskell syntax: omit single quotes from QUOTEMARKS

https://gerrit.wikimedia.org/r/86395
Comment 3 Gerrit Notification Bot 2013-10-24 05:51:05 UTC
Change 86395 merged by Adamw:
Haskell syntax: omit single quotes from QUOTEMARKS

https://gerrit.wikimedia.org/r/86395
Comment 4 Daniel Mlot 2013-10-25 04:36:43 UTC
Thank you for the patch! Looking forward to its arrival at Wikibooks.
Comment 5 Matěj Grabovský 2014-05-11 18:08:58 UTC
The change has caused incorrect rendering of double quotes in Char literals. For example, see second snippet in https://en.wikibooks.org/w/index.php?title=Write_Yourself_a_Scheme_in_48_Hours/Parsing&oldid=2627491#Return_Values
Comment 6 Daniel Mlot 2014-05-13 04:33:20 UTC
Indeed; though we exchanged an issue which shown up often for a less frequently seen one the situation is not ideal. To fix this issue and a few other annoyances, I am working on patches to haskell.php; they will be submitted uptsream.
Comment 7 Sam Reed (reedy) 2014-07-30 18:35:20 UTC
(In reply to Daniel Mlot from comment #6)
> Indeed; though we exchanged an issue which shown up often for a less
> frequently seen one the situation is not ideal. To fix this issue and a few
> other annoyances, I am working on patches to haskell.php; they will be
> submitted uptsream.

It looks like these have been done? https://github.com/GeSHi/geshi-1.0/blob/master/src/geshi/haskell.php :)
Comment 8 Gerrit Notification Bot 2014-07-30 18:37:45 UTC
Change 150622 had a related patch set uploaded by Reedy:
Update GeSHi from upstream

https://gerrit.wikimedia.org/r/150622
Comment 9 Derk-Jan Hartman 2014-11-02 13:45:22 UTC
The above patchset was merged and has actually undone this change in our core, but did bring multiple other enhancements.

I have created an upstream pull request: https://github.com/GeSHi/geshi-1.0/pull/30 for this particular problem though.
Comment 10 Gerrit Notification Bot 2014-11-02 14:03:16 UTC
Change 150622 merged by jenkins-bot:
Update GeSHi from upstream

https://gerrit.wikimedia.org/r/150622

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links