Update classify.rst - clarify that <true> is boolean, not string#148
Update classify.rst - clarify that <true> is boolean, not string#148philipmgrant wants to merge 1 commit intoisc-projects:masterfrom
Conversation
…'true' In the "list of classification values" table, change the example for boolean values from the quoted string 'true' to the formatted boolean literal ``true``
|
Not convinced there is something to fix here: values are represented as strings so The grammar has two entries: one returning a boolean, the second a string. There are tokens to bridge between expressions of the two types:
For the token Of course here I assume that all expressions are produced by the parser and not built directly from tokens but this is the case at the exception of a few unit test. |
|
Here's the specific scenario that prompted me to make the PR. I tried to write a class definition like this: because, based on the documentation, I assumed that the But that gave an error, and I had to write: That's a better syntax, I'm not complaining about that! But it seems like there really is a difference between an expression that returns a boolean (as |
|
BTW in |
In the "list of classification values" table, the examples for boolean-typed values are shown as the single-quoted string 'true'. Change this to the formatted boolean literal
true.