SoftwareIntrospectionLab/guilty — pull request #3 by linzhp (2011-10-24 06:10:50+00:00)
Title: handler author names with paretheses

Problem:

When encountering such blame lines:

1f2a4b37accc023c96148d54d8daa3a751446c44 railties/lib/initializer.rb (Hongli Lai (Phusion 1209667629 +0200 31) if defined?(RAILS_ROOT)
1f2a4b37accc023c96148d54d8daa3a751446c44 railties/lib/initializer.rb (Hongli Lai (Phusion 1209667629 +0200 32) RAILS_ROOT
1f2a4b37accc023c96148d54d8daa3a751446c44 railties/lib/initializer.rb (Hongli Lai (Phusion 1209667629 +0200 33) else
1f2a4b37accc023c96148d54d8daa3a751446c44 railties/lib/initializer.rb (Hongli Lai (Phusion 1209667629 +0200 34) nil
1f2a4b37accc023c96148d54d8daa3a751446c44 railties/lib/initializer.rb (Hongli Lai (Phusion 1209667629 +0200 35) end

Old pattern would parse "Phusion" as author, which is wrong, and end up not being able to find the author_id. The new pattern would consider the string after the first left parenthesis as author. The drawback is if there is any left parenthesis in file path, the author name would be wrong again. Hope there won't be too many such file paths. Will see