Symbol import: preemptively error on non-utf8 symbols
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Feb 26, 8:49 PM

	Unknown Object (File)
	Mon, Feb 24, 11:32 AM

	Unknown Object (File)
	Sat, Feb 22, 7:11 PM

	Unknown Object (File)
	Sat, Feb 22, 7:11 PM

	Unknown Object (File)
	Sat, Feb 22, 7:11 PM

	Unknown Object (File)
	Sat, Feb 22, 6:52 PM

	Unknown Object (File)
	Sat, Feb 15, 9:34 AM

	Unknown Object (File)
	Sat, Feb 15, 9:34 AM

Subscribers

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...

epriestley accepted this revision.Feb 26 2014, 8:41 PM

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

Revision Contents
Changeset List

Path

Size

scripts/

symbols/

import_project_symbols.php

45 lines

Diff 19853

View Options

scripts/symbols/import_project_symbols.php

Symbol import: preemptively error on non-utf8 symbolsClosedPublicActions