The idea is that to attempt a regex match. When there is a regex match, use the exact match and count additions and removals and do a diff to determine if it the pattern is a match. This allows one to have a pattern like this: more_in_code: r'RefCountedThreadSafe<|RefCounted<' and then the pattern matches if the number of ThreadSafeRefCount< or RefCounted< increased in any file (even if one was replaced by the other). This is a simple way to do mimic an intraline diff and say that the an occurrence of a string is new (which is typically what people would want to see).
Created attachment 109378 [details] Patch
Comment on attachment 109378 [details] Patch Ok. My sense is you should wire this up to a real bot pretty soon so we can get experience. Ship first, ask questions later!
fwiw, I think this one is by far more useful than added_in_lines (and I may remove "added/remove_in_line" in the future).
(In reply to comment #3) > fwiw, I think this one is by far more useful than added_in_lines (and I may remove "added/remove_in_line" in the future). Oh definitely. This looks very cool. I still think you should ship early and often. :)
Comment on attachment 109378 [details] Patch Clearing flags on attachment: 109378 Committed r96441: <http://trac.webkit.org/changeset/96441>
All reviewed patches have been landed. Closing bug.