Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix documentation typo #22851

Open
wants to merge 1 commit into
base: blead
Choose a base branch
from

Conversation

2shortplanks
Copy link
Contributor

Simple documentation typo fix

@@ -134,7 +134,7 @@ bundles.
$val = false;
Returns the boolean fiction value. While any non-true scalar value is
Returns the boolean false value. While any non-true scalar value is
Copy link
Member

Choose a reason for hiding this comment

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

I think the documentation for true above should be changed to match too, i.e. "the boolean true value."

@johannessen
Copy link
Contributor

I don’t think this is a typo. It looks like just an artistic choice of words.

@jkeenan
Copy link
Contributor

jkeenan commented Dec 11, 2024

I don’t think this is a typo. It looks like just an artistic choice of words.

@leonerd, the terminology "boolean fiction value" was introduced in 6a2e756. Would you care to comment on this?

@khwilliamson
Copy link
Contributor

Even if it is artistic, if its meaning isn't obvious, it should be changed. I kinda see what it means, and I'm a native English speaker.

@johannessen
Copy link
Contributor

It’s nice for Perl documentation to have some personality.

true (“boolean truth”) is right above false (“boolean fiction”) in builtin. I personally think reading it in context like that makes the meaning very obvious. But I already know the meaning, so I’m the best person to judge that. And if it’s not obvious, it should indeed be changed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants