Page MenuHomePhabricator

Symbol import: preemptively error on non-utf8 symbols
ClosedPublic

Authored by avivey on Feb 26 2014, 8:35 PM.

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP361e49dd6799: Symbol import: preemptively error on non-utf8 symbols
Summary

Put a test somewhere we can --ignore it.

Also fix path tests.

Test Plan

insert good/bad values at various positions.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

The only reason I actually got a non-utf symbol was because of a ctags bug; But seeing as that bug is open since 2007...

Closed by commit rP361e49dd6799 (authored by @avivey, committed by @epriestley).