Quote parser broken
Author
Discussion

Pigeon

Original Poster:

18,535 posts

275 months

Saturday 13th June 2009
quotequote all
Putting a non-PHcode-reserved word inside square brackets inside a quote exposes what looks like a greedy regexp bug smile


See here:

http://www.pistonheads.com/gassing/topic.asp?h=0&a...

I replaced "they" in the quote from Total Control with "[superchargers]" in the interests of clarity, and it broke the quote...

Have you got .* where it ought to be [^]]* smile

Mr Will

13,719 posts

235 months

Monday 15th June 2009
quotequote all
Good spot!

We have a new parser in testing at the moment, due to go live very shortly. I'll check that the new parser fixes the bug and if not I'll make the necessary changes.

thumbup

Pigeon

Original Poster:

18,535 posts

275 months

Tuesday 16th June 2009
quotequote all
Cool thumbup

him_over_there

970 posts

235 months

Wednesday 17th June 2009
quotequote all
Can you also fix the code tags ?

some code


[NOWIKI]

???

Edited by him_over_there on Wednesday 17th June 11:57

him_over_there

970 posts

235 months

Wednesday 17th June 2009
quotequote all
You cant even put code in the code tags

{{{

int myCFunction(int x)
{
/* do something */
}

}}}

RacingPete

9,197 posts

233 months

Wednesday 17th June 2009
quotequote all
him_over_there said:
You cant even put code in the code tags

{{{

int myCFunction(int x)
{
/* do something */
}

}}}
That is fixed in the new parser... plus you can put as many code tags as you like.

Due for release early July at the moment.