Page MenuHomePhabricator

Tokenizers - improve algorithm around exact matches.
ClosedPublic

Authored by btrahan on Jan 9 2015, 10:43 PM.
Tags
None
Referenced Files
F18741937: D11305.id.diff
Thu, Oct 2, 4:37 PM
F18729312: D11305.id.diff
Tue, Sep 30, 10:59 AM
F18713575: D11305.diff
Mon, Sep 29, 6:35 AM
F18678475: D11305.id27163.diff
Thu, Sep 25, 10:43 PM
F18671668: D11305.id27164.diff
Thu, Sep 25, 4:46 AM
F18627438: D11305.id27163.diff
Tue, Sep 16, 4:00 AM
F18621260: D11305.diff
Mon, Sep 15, 7:31 AM
F18594572: D11305.diff
Fri, Sep 12, 6:38 PM
Subscribers

Details

Summary

Fixes T6102. Gets the from D10867 running in the tokenizer. This thing is pretty copy / pastey, but I guess that's okay?

Test Plan

looked at a project tokenizer and typed "project". since I have a million things with the word "project" in it, I was delighted to see the "project" project first in this project tokenizer.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Tokenizers - improve algorithm around exact matches..
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
webroot/rsrc/js/core/Prefab.js
105–107

whoops, will fix

forgot the "/g" on the quick regex I wrote in my text editor to update the iteration variables.

epriestley edited edge metadata.

yeah fix all that extra jj stuff

seems fine otherwise

This revision is now accepted and ready to land.Jan 9 2015, 10:52 PM

(Your update is fine, I was just looking at v1.)

This revision was automatically updated to reflect the committed changes.