Skip to content

Implemented try methods.#335

Closed
AdamSpeight2008 wants to merge 1 commit intodotnet:masterfrom
AdamSpeight2008:AdamSpeight2008
Closed

Implemented try methods.#335
AdamSpeight2008 wants to merge 1 commit intodotnet:masterfrom
AdamSpeight2008:AdamSpeight2008

Conversation

@AdamSpeight2008
Copy link
Contributor

An implementation for #334

- TryPeekChar
- TryGetChar
- TryPeekAheadChar
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This could be include into the while condition

While TryPeekAheadChar(Here,ch) AndAlso (Not IsHexDigit(ch))
  Here +=1
End While

Copy link
Contributor

Choose a reason for hiding this comment

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

While TryPeekAheadChar(Here, ch) AndAlso IsHexDigit(ch)

The condition is that it is a hex digit. +1 on inlining into the condition.

@AdamSpeight2008 AdamSpeight2008 deleted the AdamSpeight2008 branch February 9, 2015 23:00
@AdamSpeight2008 AdamSpeight2008 restored the AdamSpeight2008 branch February 9, 2015 23:04
@AdamSpeight2008
Copy link
Contributor Author

Updated PR #342

@theoy theoy added Area-Compilers Concept-API This issue involves adding, removing, clarification, or modification of an API. Enhancement and removed Concept-API This issue involves adding, removing, clarification, or modification of an API. labels Feb 10, 2015
Copy link
Contributor

Choose a reason for hiding this comment

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

Inline to condition.

@gafter gafter added the Resolution-Won't Fix A real bug, but Triage feels that the issue is not impactful enough to spend time on. label Dec 9, 2015
dibarbet pushed a commit to dibarbet/roslyn that referenced this pull request Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Compilers Feature Request Resolution-Won't Fix A real bug, but Triage feels that the issue is not impactful enough to spend time on.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants