Skip to content

[Compiler VB Scanner] Peep functionality. - #6295

Closed
AdamSpeight2008 wants to merge 1 commit into
dotnet:masterfrom
AdamSpeight2008:Scanner(Peep)
Closed

[Compiler VB Scanner] Peep functionality.#6295
AdamSpeight2008 wants to merge 1 commit into
dotnet:masterfrom
AdamSpeight2008:Scanner(Peep)

Conversation

@AdamSpeight2008

Copy link
Copy Markdown
Contributor

This adds peep functionality into the scanner, which is combination of CanGet and Peek. It does not alter the existing content of the out parameter (ch) when CanGet is false.

NOTE: This is different mechinism to TryParse methods, this is deliberate so that existing value can be used in subsequent conditions.

Usage as been implement into ScannerInterpolatedString

This adds peep functionality into the scanner, which is combination of CanGet and Peek. It does not alter the existing content of the out parameter (ch) when CanGet is false.

NOTE: This is different mechinism to TryParse methods, this is deliberate so that existing value can be used in subsequent conditions.
@AdamSpeight2008

Copy link
Copy Markdown
Contributor Author

@dotnet-bot retest this please.

@AdamSpeight2008

Copy link
Copy Markdown
Contributor Author

Integrated into #6305

@gafter gafter added the Resolution-Duplicate The described behavior is tracked in another issue label Dec 9, 2015
@AdamSpeight2008
AdamSpeight2008 deleted the Scanner(Peep) branch April 11, 2018 20:36
github-actions Bot pushed a commit that referenced this pull request Apr 1, 2025
* Implement Use 'StartsWith' instead of 'IndexOf' analyzer

* Support fix all

* Write a test

* address feedback, more tests

* Remove unnecessary using

* Simplify

* Refactor

* wip

* Redundant comment

* Remove unused using directive

* Handle some scenarios, fix tests, and add more tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Compilers cla-already-signed Resolution-Duplicate The described behavior is tracked in another issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants