Why <math> element handler removes double round brackets?

[skalee]

See line 13 in <math> element converter:

reverse_adoc/lib/reverse_adoc/converters/math.rb

Lines 10 to 15 in 99ec3a1

	def convert(node, state = {})
	stem = node.to_s.gsub(/\n/, " ")
	stem = MathML2AsciiMath.m2a(stem) if ReverseAdoc.config.mathml2asciimath
	stem = stem.gsub(/\[/, "\\[").gsub(/\]/, "\\]").gsub(/\(\(([^\)]+)\)\)/, "(\\1)") unless stem.nil?
	" stem:[" << stem << "] "
	end

Last #gsub call in that line replaces doubled round brackets (like in ((...))) with single ones ((...)). What's the reason for doing that? Although it seems not very harmful, it actually changes the expression itself and the way it is rendered with MathJax. For example:

@opoudjis, this change was made by you 3 years ago in e16d77d, do you remember why?

[opoudjis]

I don't, but you should not trust the MathML generated by Microsoft's XSLT (exported from source OOXML). I would not at all be surprised if Microsoft's XSLT overgenerates brackets, and I clamped down on it doing so in response three years ago.

I'd trial it with some actual Word exports before commiting any such change if I were you.

[skalee]

Okay. I'll make it optional if I stumble upon any issues caused by this thing.