User talk:Paxed/MixedSyntaxHighlight - Revision history

Ilmari Karonen: don't see why it would

2010-12-19T10:51:58Z

don't see why it would

Paxed: /* Bug: regexp eats leading spaces */ i'll look into it later.

2010-12-19T10:30:38Z

‎Bug: regexp eats leading spaces: i'll look into it later.

Ilmari Karonen: /* Bug: regexp eats leading spaces */ suggest fix

2010-12-19T09:42:44Z

‎Bug: regexp eats leading spaces: suggest fix

Paxed: /* Bug: regexp eats leading spaces */

2010-12-19T08:15:31Z

‎Bug: regexp eats leading spaces

Ilmari Karonen: Bug: regexp eats leading spaces

2010-12-19T07:10:00Z

Bug: regexp eats leading spaces

New page

== Bug: regexp eats leading spaces ==

The regexp <code>/^ <span id=\"line([0-9]{1,4})">[0-9]{1,4}\. *(.*)<\/span>$/</code> causes indentation using spaces to disappear; see e.g. [[Source:Pray.c#dosacrifice]]. Presumably, removing the first "*" (or replacing it with "?") would fix the problem. --[[User:Ilmari Karonen|Ilmari Karonen]] 07:10, 19 December 2010 (UTC)

← Older revision		Revision as of 10:51, 19 December 2010
Line 5:		Line 5:
	::Hmm, yeah, that's tricky... maybe you could make the regexp <code>/^ <span id=\"line([0-9]+)">[ 0]\1\.( )(.*)<\/span>$/</code> and do something like <code>$text .= substr($matches[2], 5 - strlen($matches[1])) . $matches[3] . "\n";</code>? That should handle both left- and right-adjusted line numbers (and, coincidentally, should reject any lines where the number doesn't match the ID). --[[User:Ilmari Karonen\|Ilmari Karonen]] 09:42, 19 December 2010 (UTC)		::Hmm, yeah, that's tricky... maybe you could make the regexp <code>/^ <span id=\"line([0-9]+)">[ 0]\1\.( )(.*)<\/span>$/</code> and do something like <code>$text .= substr($matches[2], 5 - strlen($matches[1])) . $matches[3] . "\n";</code>? That should handle both left- and right-adjusted line numbers (and, coincidentally, should reject any lines where the number doesn't match the ID). --[[User:Ilmari Karonen\|Ilmari Karonen]] 09:42, 19 December 2010 (UTC)
	:::I'll look into it later today; hopefully this doesn't cause too much cpu usage increase... --[[User:Paxed\|paxed]] 10:30, 19 December 2010 (UTC)		:::I'll look into it later today; hopefully this doesn't cause too much cpu usage increase... --[[User:Paxed\|paxed]] 10:30, 19 December 2010 (UTC)
		+	::::I don't see why it would. (If you want to optimize the CPU usage, I'd suggest collecting the pieces of text in an array and doing a single join() to concatenate them at the end.) --[[User:Ilmari Karonen\|Ilmari Karonen]] 10:51, 19 December 2010 (UTC)

← Older revision		Revision as of 10:30, 19 December 2010
Line 4:		Line 4:
	:The problem is that lines have the same number of characters for the line numbers... but lines with fewer numbers are padded with spaces. --[[User:Paxed\|paxed]] 08:15, 19 December 2010 (UTC)		:The problem is that lines have the same number of characters for the line numbers... but lines with fewer numbers are padded with spaces. --[[User:Paxed\|paxed]] 08:15, 19 December 2010 (UTC)
	::Hmm, yeah, that's tricky... maybe you could make the regexp <code>/^ <span id=\"line([0-9]+)">[ 0]\1\.( )(.*)<\/span>$/</code> and do something like <code>$text .= substr($matches[2], 5 - strlen($matches[1])) . $matches[3] . "\n";</code>? That should handle both left- and right-adjusted line numbers (and, coincidentally, should reject any lines where the number doesn't match the ID). --[[User:Ilmari Karonen\|Ilmari Karonen]] 09:42, 19 December 2010 (UTC)		::Hmm, yeah, that's tricky... maybe you could make the regexp <code>/^ <span id=\"line([0-9]+)">[ 0]\1\.( )(.*)<\/span>$/</code> and do something like <code>$text .= substr($matches[2], 5 - strlen($matches[1])) . $matches[3] . "\n";</code>? That should handle both left- and right-adjusted line numbers (and, coincidentally, should reject any lines where the number doesn't match the ID). --[[User:Ilmari Karonen\|Ilmari Karonen]] 09:42, 19 December 2010 (UTC)
		+	:::I'll look into it later today; hopefully this doesn't cause too much cpu usage increase... --[[User:Paxed\|paxed]] 10:30, 19 December 2010 (UTC)

← Older revision		Revision as of 09:42, 19 December 2010
Line 3:		Line 3:
	The regexp <code>/^ <span id=\"line([0-9]{1,4})">[0-9]{1,4}\. (.)<\/span>$/</code> causes indentation using spaces to disappear; see e.g. [[Source:Pray.c#dosacrifice]]. Presumably, removing the first "*" (or replacing it with "?") would fix the problem. --[[User:Ilmari Karonen\|Ilmari Karonen]] 07:10, 19 December 2010 (UTC)		The regexp <code>/^ <span id=\"line([0-9]{1,4})">[0-9]{1,4}\. (.)<\/span>$/</code> causes indentation using spaces to disappear; see e.g. [[Source:Pray.c#dosacrifice]]. Presumably, removing the first "*" (or replacing it with "?") would fix the problem. --[[User:Ilmari Karonen\|Ilmari Karonen]] 07:10, 19 December 2010 (UTC)
	:The problem is that lines have the same number of characters for the line numbers... but lines with fewer numbers are padded with spaces. --[[User:Paxed\|paxed]] 08:15, 19 December 2010 (UTC)		:The problem is that lines have the same number of characters for the line numbers... but lines with fewer numbers are padded with spaces. --[[User:Paxed\|paxed]] 08:15, 19 December 2010 (UTC)
		+	::Hmm, yeah, that's tricky... maybe you could make the regexp <code>/^ <span id=\"line([0-9]+)">[ 0]\1\.( )(.*)<\/span>$/</code> and do something like <code>$text .= substr($matches[2], 5 - strlen($matches[1])) . $matches[3] . "\n";</code>? That should handle both left- and right-adjusted line numbers (and, coincidentally, should reject any lines where the number doesn't match the ID). --[[User:Ilmari Karonen\|Ilmari Karonen]] 09:42, 19 December 2010 (UTC)

← Older revision		Revision as of 08:15, 19 December 2010
Line 2:		Line 2:

	The regexp <code>/^ <span id=\"line([0-9]{1,4})">[0-9]{1,4}\. (.)<\/span>$/</code> causes indentation using spaces to disappear; see e.g. [[Source:Pray.c#dosacrifice]]. Presumably, removing the first "*" (or replacing it with "?") would fix the problem. --[[User:Ilmari Karonen\|Ilmari Karonen]] 07:10, 19 December 2010 (UTC)		The regexp <code>/^ <span id=\"line([0-9]{1,4})">[0-9]{1,4}\. (.)<\/span>$/</code> causes indentation using spaces to disappear; see e.g. [[Source:Pray.c#dosacrifice]]. Presumably, removing the first "*" (or replacing it with "?") would fix the problem. --[[User:Ilmari Karonen\|Ilmari Karonen]] 07:10, 19 December 2010 (UTC)
		+	:The problem is that lines have the same number of characters for the line numbers... but lines with fewer numbers are padded with spaces. --[[User:Paxed\|paxed]] 08:15, 19 December 2010 (UTC)