This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author serhiy.storchaka
Recipients martin.panter, serhiy.storchaka
Date 2015-11-19.07:57:07
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1447919827.84.0.976832879983.issue25663@psf.upfronthosting.co.za>
In-reply-to
Content
Nice work.

An alternative approach is to make "matches" a dict. And instead of

    if match not in seen:
        seen.add(word)
        matches.append(match)

use just

    matches[word] = match

I don't know what approach is better.

Added other minor comments on Rietveld.
History
Date User Action Args
2015-11-19 07:57:07serhiy.storchakasetrecipients: + serhiy.storchaka, martin.panter
2015-11-19 07:57:07serhiy.storchakasetmessageid: <1447919827.84.0.976832879983.issue25663@psf.upfronthosting.co.za>
2015-11-19 07:57:07serhiy.storchakalinkissue25663 messages
2015-11-19 07:57:07serhiy.storchakacreate
Morty Proxy This is a proxified and sanitized view of the page, visit original site.