Skip to content

syntax: add benchmarks for scanning and parsing#238

Open
alandonovan wants to merge 1 commit into
masterfrom
scan-parse-benchmarks
Open

syntax: add benchmarks for scanning and parsing#238
alandonovan wants to merge 1 commit into
masterfrom
scan-parse-benchmarks

Conversation

@alandonovan
Copy link
Copy Markdown
Contributor

No description provided.

Change-Id: Ie1269b8899c4454dc955c7453ed9e9266aceaaa3
@alandonovan alandonovan requested a review from jayconrod August 21, 2019 19:24
Comment thread syntax/parse.go

// ScanAndDiscard tokenizes the input data and discards the tokens.
// Parameters are as for Parse.
// It exists only for internal benchmarking purposes.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's unfortunate this needs to be part of the external API for benchmarking. Could the "scan" benchmark be moved to the syntax package instead?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants