Filing an issue on this as I may not get to this today and do not want to forget about it.
|
/// <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:
|
case 'n': |
|
case 'N': |
|
suffix |= NumberSuffixFlags.BigInteger; |
|
break; |
Small thing, but might lead to confusion later on. 🙂
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
This comment is incorrect; the suffix used for BigInteger is actually
Nas can be seen here in the tokenizer code:PowerShell/src/System.Management.Automation/engine/parser/tokenizer.cs
Lines 4021 to 4024 in 540737e
Small thing, but might lead to confusion later on. 🙂