Skip to content
This repository was archived by the owner on Mar 8, 2020. It is now read-only.

Fix Unicode positions #40

Merged
dennwc merged 1 commit into from
May 30, 2019
Merged

Fix Unicode positions #40

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions driver/fixtures/fixtures_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,14 @@ var Suite = &fixtures.Suite{
"ASTInclusionStatement",
},
},
// TODO: This won't really work because of the C++ macros expansion.
// The native AST includes an expanded macros as a token
// but we expect the token to match the source file exactly.
//VerifyTokens: []positioner.VerifyToken{
// {Types: []string{
// "CPPASTLiteralExpression",
// }},
//},
}

func TestCppDriver(t *testing.T) {
Expand Down
2 changes: 1 addition & 1 deletion driver/normalizer/normalizer.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ var Preprocess = Transformers([][]Transformer{
}...)

var PreprocessCode = []CodeTransformer{
positioner.FromOffset(),
positioner.FromUTF16Offset(),
}

var Normalize = Transformers([][]Transformer{
Expand Down
33,246 changes: 16,623 additions & 16,623 deletions fixtures/cpp17_2.cpp.sem.uast
View file
Open in desktop

Large diffs are not rendered by default.

37,158 changes: 18,579 additions & 18,579 deletions fixtures/cpp17_2.cpp.uast
View file
Open in desktop

Large diffs are not rendered by default.

Loading