ruby-core@ruby-lang.org archive (unofficial mirror)
 help / color / mirror / Atom feed
* [ruby-core:59314] [ruby-trunk - Bug #9299][Open] Required keyowrd arguments and arity
@ 2013-12-25 21:33 marcandre (Marc-Andre Lafortune)
  2014-02-22  6:57 ` [ruby-core:60991] [Backport21 - Backport #9299] [Feedback] " naruse
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: marcandre (Marc-Andre Lafortune) @ 2013-12-25 21:33 UTC (permalink / raw
  To: ruby-core


Issue #9299 has been reported by marcandre (Marc-Andre Lafortune).

----------------------------------------
Bug #9299: Required keyowrd arguments and arity
https://bugs.ruby-lang.org/issues/9299

Author: marcandre (Marc-Andre Lafortune)
Status: Open
Priority: Normal
Assignee: marcandre (Marc-Andre Lafortune)
Category: core
Target version: 
ruby -v: 2.1.0
Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN, 2.1: UNKNOWN


While fixing #8072, I noticed another bug: a required keyword argument should add 1 to the arity:

    proc{|required:|}.arity # => 0, should be 1


-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:60991] [Backport21 - Backport #9299] [Feedback] Required keyowrd arguments and arity
  2013-12-25 21:33 [ruby-core:59314] [ruby-trunk - Bug #9299][Open] Required keyowrd arguments and arity marcandre (Marc-Andre Lafortune)
@ 2014-02-22  6:57 ` naruse
  2014-02-28  2:15 ` [ruby-core:61131] [Backport21 - Backport #9299] " ruby-core
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: naruse @ 2014-02-22  6:57 UTC (permalink / raw
  To: ruby-core

Issue #9299 has been updated by Yui NARUSE.

Status changed from Open to Feedback
Assignee changed from Yui NARUSE to Marc-Andre Lafortune

Like #8072, this is not a simple backport.
Please get confirmation by matz

----------------------------------------
Backport #9299: Required keyowrd arguments and arity
https://bugs.ruby-lang.org/issues/9299#change-45392

* Author: Marc-Andre Lafortune
* Status: Feedback
* Priority: Normal
* Assignee: Marc-Andre Lafortune
----------------------------------------
While fixing #8072, I noticed another bug: a required keyword argument should add 1 to the arity:

    proc{|required:|}.arity # => 0, should be 1



-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:61131] [Backport21 - Backport #9299] Required keyowrd arguments and arity
  2013-12-25 21:33 [ruby-core:59314] [ruby-trunk - Bug #9299][Open] Required keyowrd arguments and arity marcandre (Marc-Andre Lafortune)
  2014-02-22  6:57 ` [ruby-core:60991] [Backport21 - Backport #9299] [Feedback] " naruse
@ 2014-02-28  2:15 ` ruby-core
  2014-02-28  3:01 ` [ruby-core:61133] " matz
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: ruby-core @ 2014-02-28  2:15 UTC (permalink / raw
  To: ruby-core

Issue #9299 has been updated by Marc-Andre Lafortune.

Assignee changed from Marc-Andre Lafortune to Yukihiro Matsumoto

Please excuse my ignorance, I'm not sure what a simple backport is.

Dear Matz, hopefully you will know.

I always put my bug fixes as backport requests, but I welcome guidelines.


----------------------------------------
Backport #9299: Required keyowrd arguments and arity
https://bugs.ruby-lang.org/issues/9299#change-45512

* Author: Marc-Andre Lafortune
* Status: Feedback
* Priority: Normal
* Assignee: Yukihiro Matsumoto
----------------------------------------
While fixing #8072, I noticed another bug: a required keyword argument should add 1 to the arity:

    proc{|required:|}.arity # => 0, should be 1



-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:61133] [Backport21 - Backport #9299] Required keyowrd arguments and arity
  2013-12-25 21:33 [ruby-core:59314] [ruby-trunk - Bug #9299][Open] Required keyowrd arguments and arity marcandre (Marc-Andre Lafortune)
  2014-02-22  6:57 ` [ruby-core:60991] [Backport21 - Backport #9299] [Feedback] " naruse
  2014-02-28  2:15 ` [ruby-core:61131] [Backport21 - Backport #9299] " ruby-core
@ 2014-02-28  3:01 ` matz
  2014-03-01  8:25 ` [ruby-core:61166] [Backport21 - Backport #9299] [Closed] " naruse
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: matz @ 2014-02-28  3:01 UTC (permalink / raw
  To: ruby-core

Issue #9299 has been updated by Yukihiro Matsumoto.


I consider this as a bug.  So it should be back-ported to 2.1, I think.

Matz.


----------------------------------------
Backport #9299: Required keyowrd arguments and arity
https://bugs.ruby-lang.org/issues/9299#change-45514

* Author: Marc-Andre Lafortune
* Status: Feedback
* Priority: Normal
* Assignee: Yukihiro Matsumoto
----------------------------------------
While fixing #8072, I noticed another bug: a required keyword argument should add 1 to the arity:

    proc{|required:|}.arity # => 0, should be 1



-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:61166] [Backport21 - Backport #9299] [Closed] Required keyowrd arguments and arity
  2013-12-25 21:33 [ruby-core:59314] [ruby-trunk - Bug #9299][Open] Required keyowrd arguments and arity marcandre (Marc-Andre Lafortune)
                   ` (2 preceding siblings ...)
  2014-02-28  3:01 ` [ruby-core:61133] " matz
@ 2014-03-01  8:25 ` naruse
  2014-03-01  8:46 ` [ruby-core:61167] [Backport21 - Backport #9299] " naruse
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: naruse @ 2014-03-01  8:25 UTC (permalink / raw
  To: ruby-core

Issue #9299 has been updated by Yui NARUSE.

Status changed from Feedback to Closed
% Done changed from 0 to 100

Applied in changeset r45226.

----------
merge revision(s) 44433: [Backport #9299]

	* proc.c: Having any mandatory keyword argument increases min arity
	  [#9299]

----------------------------------------
Backport #9299: Required keyowrd arguments and arity
https://bugs.ruby-lang.org/issues/9299#change-45531

* Author: Marc-Andre Lafortune
* Status: Closed
* Priority: Normal
* Assignee: Yukihiro Matsumoto
----------------------------------------
While fixing #8072, I noticed another bug: a required keyword argument should add 1 to the arity:

    proc{|required:|}.arity # => 0, should be 1



-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:61167] [Backport21 - Backport #9299] Required keyowrd arguments and arity
  2013-12-25 21:33 [ruby-core:59314] [ruby-trunk - Bug #9299][Open] Required keyowrd arguments and arity marcandre (Marc-Andre Lafortune)
                   ` (3 preceding siblings ...)
  2014-03-01  8:25 ` [ruby-core:61166] [Backport21 - Backport #9299] [Closed] " naruse
@ 2014-03-01  8:46 ` naruse
  2014-03-02  2:14 ` [ruby-core:61213] [Backport21 - Backport #9299] [Open] " ruby-core
  2017-02-09 11:59 ` [ruby-core:79486] [Backport21 Backport#9299] " romaseo86
  6 siblings, 0 replies; 8+ messages in thread
From: naruse @ 2014-03-01  8:46 UTC (permalink / raw
  To: ruby-core

Issue #9299 has been updated by Yui NARUSE.


Thank you for comments.

Marc-Andre Lafortune wrote:
> Please excuse my ignorance, I'm not sure what a simple backport is.
> 
> Dear Matz, hopefully you will know.
> 
> I always put my bug fixes as backport requests, but I welcome guidelines.

yeah, I know you understand backport policy.
What I wanted to say is "I don't understand Proc#arity's keyword argument spec and can't judge whether spec change or bug fix".
(I accidentally backported #8072 related another backport, but the main issue is the spec of arity for keyword arguments)

Therefore could you describe the spec to Proc#arity's rdoc?
I want to backport it to Ruby 2.1.2.


Anyway I had written HowToRequestBackport.
The use of Backport custom field will help you.
https://bugs.ruby-lang.org/projects/ruby/wiki/HowToRequestBackport

----------------------------------------
Backport #9299: Required keyowrd arguments and arity
https://bugs.ruby-lang.org/issues/9299#change-45532

* Author: Marc-Andre Lafortune
* Status: Closed
* Priority: Normal
* Assignee: Yukihiro Matsumoto
----------------------------------------
While fixing #8072, I noticed another bug: a required keyword argument should add 1 to the arity:

    proc{|required:|}.arity # => 0, should be 1



-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:61213] [Backport21 - Backport #9299] [Open] Required keyowrd arguments and arity
  2013-12-25 21:33 [ruby-core:59314] [ruby-trunk - Bug #9299][Open] Required keyowrd arguments and arity marcandre (Marc-Andre Lafortune)
                   ` (4 preceding siblings ...)
  2014-03-01  8:46 ` [ruby-core:61167] [Backport21 - Backport #9299] " naruse
@ 2014-03-02  2:14 ` ruby-core
  2017-02-09 11:59 ` [ruby-core:79486] [Backport21 Backport#9299] " romaseo86
  6 siblings, 0 replies; 8+ messages in thread
From: ruby-core @ 2014-03-02  2:14 UTC (permalink / raw
  To: ruby-core

Issue #9299 has been updated by Marc-Andre Lafortune.

Status changed from Closed to Open

Naruse-san: You're right, the rdoc of Proc#arity was quite out of date. I've committed r45236-r45238 (04042ec6eb4..47d6a7ee1c8) to address this.

Matz: would you have time to review the rdoc of Proc#arity, especially the part about the different treatment of lambdas vs procs?

Thanks

----------------------------------------
Backport #9299: Required keyowrd arguments and arity
https://bugs.ruby-lang.org/issues/9299#change-45565

* Author: Marc-Andre Lafortune
* Status: Open
* Priority: Normal
* Assignee: Yukihiro Matsumoto
----------------------------------------
While fixing #8072, I noticed another bug: a required keyword argument should add 1 to the arity:

    proc{|required:|}.arity # => 0, should be 1



-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [ruby-core:79486] [Backport21 Backport#9299] Required keyowrd arguments and arity
  2013-12-25 21:33 [ruby-core:59314] [ruby-trunk - Bug #9299][Open] Required keyowrd arguments and arity marcandre (Marc-Andre Lafortune)
                   ` (5 preceding siblings ...)
  2014-03-02  2:14 ` [ruby-core:61213] [Backport21 - Backport #9299] [Open] " ruby-core
@ 2017-02-09 11:59 ` romaseo86
  6 siblings, 0 replies; 8+ messages in thread
From: romaseo86 @ 2017-02-09 11:59 UTC (permalink / raw
  To: ruby-core

Issue #9299 has been updated by Peppe Signori.


Marc-Andre Lafortune wrote:
> While fixing #8072, I noticed another bug: a required keyword argument should add 1 to the arity:
> 
>     proc{|required:|}.arity # => 0, should be 1
http://papers-writings.net/


----------------------------------------
Backport #9299: Required keyowrd arguments and arity
https://bugs.ruby-lang.org/issues/9299#change-62932

* Author: Marc-Andre Lafortune
* Status: Open
* Priority: Normal
* Assignee: Yukihiro Matsumoto
----------------------------------------
While fixing #8072, I noticed another bug: a required keyword argument should add 1 to the arity:

    proc{|required:|}.arity # => 0, should be 1



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2017-02-09 11:30 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-25 21:33 [ruby-core:59314] [ruby-trunk - Bug #9299][Open] Required keyowrd arguments and arity marcandre (Marc-Andre Lafortune)
2014-02-22  6:57 ` [ruby-core:60991] [Backport21 - Backport #9299] [Feedback] " naruse
2014-02-28  2:15 ` [ruby-core:61131] [Backport21 - Backport #9299] " ruby-core
2014-02-28  3:01 ` [ruby-core:61133] " matz
2014-03-01  8:25 ` [ruby-core:61166] [Backport21 - Backport #9299] [Closed] " naruse
2014-03-01  8:46 ` [ruby-core:61167] [Backport21 - Backport #9299] " naruse
2014-03-02  2:14 ` [ruby-core:61213] [Backport21 - Backport #9299] [Open] " ruby-core
2017-02-09 11:59 ` [ruby-core:79486] [Backport21 Backport#9299] " romaseo86

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).