-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Closed
Labels
Issue-Code Cleanupthe issue is for cleaning up the code with no impact on functionalitythe issue is for cleaning up the code with no impact on functionalityResolution-No ActivityIssue has had no activity for 6 months or moreIssue has had no activity for 6 months or more
Description
Filing an issue on this as I may not get to this today and do not want to forget about it.
PowerShell/src/System.Management.Automation/engine/parser/tokenizer.cs
Lines 545 to 548 in 540737e
| /// <summary> | |
| /// Indicates 'I' suffix for BigInteger (arbitrarily large integer) numerals. | |
| /// </summary> | |
| BigInteger = 0x20 |
This comment is incorrect; the suffix used for BigInteger is actually N as can be seen here in the tokenizer code:
PowerShell/src/System.Management.Automation/engine/parser/tokenizer.cs
Lines 4021 to 4024 in 540737e
| case 'n': | |
| case 'N': | |
| suffix |= NumberSuffixFlags.BigInteger; | |
| break; |
Small thing, but might lead to confusion later on. 🙂
Metadata
Metadata
Assignees
Labels
Issue-Code Cleanupthe issue is for cleaning up the code with no impact on functionalitythe issue is for cleaning up the code with no impact on functionalityResolution-No ActivityIssue has had no activity for 6 months or moreIssue has had no activity for 6 months or more