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

less than awesome error message for 1 param expected in Rakudo #1126

Closed
p6rt opened this issue Jul 8, 2009 · 9 comments
Closed

less than awesome error message for 1 param expected in Rakudo #1126

p6rt opened this issue Jul 8, 2009 · 9 comments
Labels

Comments

@p6rt
Copy link

p6rt commented Jul 8, 2009

Migrated from rt.perl.org#67358 (status was 'resolved')

Searchable as RT67358$

@p6rt
Copy link
Author

p6rt commented Jul 8, 2009

From @masak

<masak> actually, I see another bug there... :)
<masak> rakudo​: say log(1, 10)
<p6eval> rakudo 70bfd5​: OUTPUT«too many arguments passed (2) - 1
params expected [...]
<masak> I don't know about you, but '1 params' does not make an
awesome message...
* masak submits a rakudobug about that

See what the standard of 'awesome error messages' is doing to us? But
I do think English grammar mandates "param" when it's only 1
parameter. :)

@p6rt
Copy link
Author

p6rt commented Jul 8, 2009

From @kyleha

On Wed, Jul 8, 2009 at 4​:50 PM, Carl Mäsak<perl6-bugs-followup@​perl.org> wrote​:

<p6eval> rakudo 70bfd5​: OUTPUT«too many arguments passed (2) - 1
params expected [...]

In this case, it might be easiest to leave the word out​:

Too many arguments passed (2) - 1 expected.

Even this looks not too bad to me​:

Too many arguments passed (1) - 0 expected.

Kyle.

@p6rt
Copy link
Author

p6rt commented Jul 8, 2009

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Jul 8, 2009

From @chromatic

On Wednesday 08 July 2009 14​:56​:11 Kyle Hasselbacher wrote​:

In this case, it might be easiest to leave the word out​:

Too many arguments passed (2) - 1 expected.

Even this looks not too bad to me​:

Too many arguments passed (1) - 0 expected.

It's Parrot behavior. It's trivial to change to "Too many params passed" or
"Too many results passed". Would that be clearer?

-- c

@p6rt
Copy link
Author

p6rt commented Jul 8, 2009

From @masak

chromatic (>), Kyle (>>)​:

In this case, it might be easiest to leave the word out​:

Too many arguments passed (2) - 1 expected.

Even this looks not too bad to me​:

Too many arguments passed (1) - 0 expected.

It's Parrot behavior.  It's trivial to change to "Too many params passed" or
"Too many results passed".  Would that be clearer?

Maybe, but the problem as described in the original ticket was the
inconsistency of '1' and 'params' with an 's'.

// Carl

@p6rt
Copy link
Author

p6rt commented Jul 9, 2009

From not.com@gmail.com

On Wed, Jul 8, 2009 at 3​:28 PM, Carl Mäsak<cmasak@​gmail.com> wrote​:

It's Parrot behavior.  It's trivial to change to "Too many params passed" or
"Too many results passed".  Would that be clearer?

Maybe, but the problem as described in the original ticket was the
inconsistency of '1' and 'params' with an 's'.

There's also the inconsistency of "arguments" vs "params"! Drop the
"params" altogether.

@p6rt
Copy link
Author

p6rt commented Jul 9, 2009

From @mathw

On Thu, Jul 9, 2009 at 1​:24 AM, yary<not.com@​gmail.com> wrote​:

On Wed, Jul 8, 2009 at 3​:28 PM, Carl Mäsak<cmasak@​gmail.com> wrote​:

It's Parrot behavior.  It's trivial to change to "Too many params passed" or
"Too many results passed".  Would that be clearer?

Maybe, but the problem as described in the original ticket was the
inconsistency of '1' and 'params' with an 's'.

There's also the inconsistency of "arguments" vs "params"! Drop the
"params" altogether.

There's also also the fact that 'param' is an abbreviation and looks
sloppy, so it should be 'parameter' vs 'argument'. Which is right? I
don't know, but I know that 'param' is wrong. PIR might use it, but
that's not the kind of thing we want to expose in error messages.

Or at least, I don't.

@p6rt
Copy link
Author

p6rt commented Jul 10, 2009

From @chromatic

On Wednesday 08 July 2009 17​:24​:47 yary wrote​:

On Wed, Jul 8, 2009 at 3​:28 PM, Carl Mäsak<cmasak@​gmail.com> wrote​:

Maybe, but the problem as described in the original ticket was the
inconsistency of '1' and 'params' with an 's'.

There's also the inconsistency of "arguments" vs "params"! Drop the
"params" altogether.

I declare that "beyond the scope of this ticket", which I have resolved as of
Parrot r39964.

-- c

@p6rt
Copy link
Author

p6rt commented Mar 9, 2010

@moritz - Status changed from 'open' to 'resolved'

@p6rt p6rt closed this as completed Mar 9, 2010
@p6rt p6rt added the Bug label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant