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)
	Sun, Dec 15, 9:33 AM

	Unknown Object (File)
	Mon, Dec 9, 12:00 PM

	Unknown Object (File)
	Fri, Dec 6, 11:16 PM

	Unknown Object (File)
	Wed, Dec 4, 11:29 PM

	Unknown Object (File)
	Sun, Dec 1, 12:57 PM

	Unknown Object (File)
	Wed, Nov 27, 12:19 AM

	Unknown Object (File)
	Wed, Nov 27, 12:19 AM

	Unknown Object (File)
	Fri, Nov 22, 1:22 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