Le 2007-09-16 à 8:49, Milian Wolff a écrit :
What do you think about that? Is this a bug in Markdown.pl ?
The answer is in my first email of this thread:
There is still a little strangeness about some test cases right now: PHP Markdown fails some tests from John's base Markdown testsuite, this because John's testsuite is expecting empty title attributes on images and/or links when no title has been specified (a bug I'd say). There's an opposite test in the PHP Markdown testsuite checking that the titles are indeed not present when not specified. I think both should be harmonised.
So basically, I believe PHP Markdown's behaviour is correct: no title attribute if no title was specified; and I made a specific test to ensure that in the PHP Markdown test suite.
One of the ideas behind MDTest is the principle of separate test suites. I deliberately left those *glitches* in Markdown.pl because I want the Markdown test suite to be in sync with John's.
Previously, I've made modifications to the Markdown Test test suite to correct things like that. When John released a new version of Markdown Test where a few things such as this were not fixed (despite the bug having been reported), I had to merge my fixes and additions to that new release. That's essentially what I wanted to avoid with MDTest: John test suite is left untouched in a separate folder and I have mine in another (PHP Markdown.mdtest) where I check what I want.
The PHP Markdown test suite has its own test for images with the sole purpose of telling me that things are fine despite what the image test of the main Markdown test suite tells me. In fact, most tests in the PHP Markdown test suite are like that: additional tests which should apply to any Markdown implementation.
Now, I'd like to make some changes to the Markdown test suite in MDTest, but I fear getting out of sync again. On the other side, the current situation certainly is confusing, so I'm not too sure what to do.
Michel Fortin [EMAIL PROTECTED] http://www.michelf.com/ _______________________________________________ Markdown-Discuss mailing list [email protected] http://six.pairlist.net/mailman/listinfo/markdown-discuss
