xolox / vim-misc

Miscellaneous auto-load Vim scripts
http://peterodding.com/code/vim/misc/
367 stars 56 forks source link

Made list#unique about 5 times faster. #1

Closed tarmack closed 13 years ago

tarmack commented 13 years ago

Hoi Peter,

Terwijl ik aan het werk was aan onze python ftplugin had ik een functie nodig om dubbele waarden uit een lijst te halen. Pas nadat ik dit had geimplementeerd dacht ik ineens: het moet wel heel gek lopen als dit niet in Peter zijn misc library zit. Dit was er ook, maar behoorlijk anders en aanzienlijk trager. Ik heb de vrijheid genomen je project te forken en een patch te maken.

Ik heb de functies op snelheid getest met een niet al te grote lijst. Al denk ik niet dat de grotte uberhaupt verschil zal maken. De volgorde van de lijst blijft gelijk net zoals jou originele implementatie.

Groeten, Bart

xolox commented 13 years ago

Hey Bart,

Bedankt voor de commit! Ik heb hem zojuist gemerged naar de online vim-misc repo. Jij of ik kan later deze week de wijziging mergen naar de vim-python-ftplugin met het volgende commando:

git pull -s subtree vim-misc master

Thanks again! Ik ga nu pitten, anders zit ik morgen -_- op het werk. Jij ook alvast slaap lekker ;-)

Hoi Peter,

Terwijl ik aan het werk was aan onze python ftplugin had ik een functie nodig om dubbele waarden uit een lijst te halen. Pas nadat ik dit had geimplementeerd dacht ik ineens: het moet wel heel gek lopen als dit niet in Peter zijn misc library zit. Dit was er ook, maar behoorlijk anders en aanzienlijk trager. Ik heb de vrijheid genomen je project te forken en een patch te maken.

Ik heb de functies op snelheid getest met een niet al te grote lijst. Al denk ik niet dat de grotte uberhaupt verschil zal maken. De volgorde van de lijst blijft gelijk net zoals jou originele implementatie.

Groeten, Bart