MediaWiki r86965 - Code Review

Repository:	MediaWiki
Revision:	< r86964‎ \| r86965 (on ViewVC)‎ \| r86966 >
Date:	18:44, 26 April 2011
Author:	thenub314
Status:	resolved (Comments)
Tags:	math
Comment:	This change adds support for the \widetilde command as mentioned in bug 26380
Modified paths:	/trunk/extensions/Math/math/texutil.ml (modified) (history) (diff)

Diff [purge]

Index: trunk/extensions/Math/math/texutil.ml
—	—	@@ -492,6 +492,7 @@
493	493	\| "\\hat" -> FUN_AR1 "\\hat "
494	494	\| "\\hline" -> LITERAL (TEX_ONLY "\\hline ")
495	495	\| "\\vline" -> LITERAL (TEX_ONLY "\\vline ")
	496	+ \| "\\widetilde" -> LITERAL (TEX_ONLY "\\widetilde ")
496	497	\| "\\widehat" -> LITERAL (TEX_ONLY "\\widehat ")
497	498	\| "\\overline" -> LITERAL (TEX_ONLY "\\overline ")
498	499	\| "\\overbrace" -> LITERAL (TEX_ONLY "\\overbrace ")

User	Flag	Date
Bawolff	inspected	19:18, 29 April 2011
Thelema314	inspected	14:09, 29 August 2011

Rev.	Commit summary	Author	Date
r96990	Revert changes to texvc that provide no test cases or examples of what they'r...	brion	19:00, 13 September 2011
r96993	MFT r96990: provisional revert of texvc changes that don't come with any test...	brion	19:07, 13 September 2011
r97001	* (bug 26380) Add support for \widetilde to <math>...	brion	21:02, 13 September 2011

#Comment by Bawolff (talk | contribs) 19:17, 29 April 2011

btw, don't forget to add to the bug report which revision you fixed the bug in.

Cheers.

#Comment by Thelema314 (talk | contribs) 14:09, 29 August 2011

Straightforward change.

#Comment by Brion VIBBER (talk | contribs) 19:03, 13 September 2011

Provisionally reverted in r96990 pending test cases.

#Comment by Brion VIBBER (talk | contribs) 21:04, 13 September 2011

Restored in r97002 and r97003 with a test case & a fix for the cases.

21:04, 13 September 2011 Brion VIBBER (talk | contribs) changed the status of r86965 [removed: reverted added: resolved]
19:03, 13 September 2011 Brion VIBBER (talk | contribs) changed the status of r86965 [removed: ok added: reverted]
06:46, 26 August 2011 Hashar (talk | contribs) changed the status of r86965 [removed: new added: ok]
23:11, 22 August 2011 Catrope (talk | contribs) changed the tags for r86965 [added: math]