Apr 2017 Meeting

calendar_today April 17, 2017 confirmation_number 75 tickets discussed Source English

DevMeeting-2017-04-17

Date: 2017/04/17 (Mon) Time: 14:00- 19:00 (JST) Place: Cookpad Inc. Headquarter Sign-up: https://ruby.connpass.com/event/53124/ Attendees: Language: mostly Japanese (sorry for non native Japanese speakers)

Please add your favorite ticket numbers you want to ask to discuss.

  • NOTE
    • Dev meeting IS NOT a decision making place. All decisions should be done at the bug tracker.
    • Dev meeting is a place we can ask Matz, nobu, nurse and other developers directly. Matz is a very busy person. Take this opportunity to ask him. If you can not attend, other attendees can ask instead of you (if attendees can understand your issue).
    • We will write a log about discussion to a file or to each ticket in English.
    • All activities are best-effort (keep in mind that most of us are volunteer developers).
    • The date, time and place is scheduled according to when/where we can reserve Matz’s time.

Agenda

  • NOTE: Write at least “ticket number/title/link” and your name (see example below). Explain details on the ticket. If you cannot attend the meeting, we appreciate a short summary because we can understand it more easily (long discussion is difficult to read, especially in a non-native language). Your motivation is also welcome.

About 2.5 timeframe

  • https://bugs.ruby-lang.org/projects/ruby-trunk/wiki/ReleaseEngineering25

Carry-over from previous meeting(s)

  • [Bug #13257] Symbol#respond_to?(:singleton_class) should be false (naruse)
  • [Misc #13163] Uncaught exceptions may not be reported when Thread#report_on_exception=true and Thread#abort_on_exception=true (naruse)
  • [Bug #13181] Unexpected line in rescue backtrace (shyouhei) what’s this?
  • [Feature #8263] Support discovering yield state of individual Fibers (shyouhei)
  • [Feature #8576] Add optimized method type for constant value methods (shyouhei)
  • [Bug #13188] Reinitialize Ruby VM. (shyouhei) New C API …?
  • [Feature #2740] Extend const_missing to pass in the nesting (shyouhei)
  • [Feature #13211] Hash#delete taking a splat (shyouhei)
  • [Bug #11567] Segmentation fault CFUNC :gets (shyouhei)
  • [Feature #13172] Method that yields object to block and returns result (shyouhei) new name candicate
  • [Feature #13224] Add FrozenError as a subclass of RuntimeError (shyouhei)
  • [Misc #13230] Better Do … while structure (shyouhei)
  • [Feature #13245] [PATCH] reject inter-thread TLS modification (shyouhei)
  • [Feature #13252] C API for creating strings without copying (shyouhei)
  • [Bug #13249] Access modifiers don’t have an effect inside class methods in Ruby >= 2.3 (shyouhei) design issue or bug?
  • [Feature #13265] TracePoint for basic operation redefinition (shyouhei)
  • [Bug #12834] prepend getting prepended even if it already exists in the ancestors chain (shyouhei)
  • [Bug #7976] TracePoint call is at call point, not call site (shyouhei)
  • [Feature #12901] Anonymous functions without scope lookup overhead (shyouhei) status?
  • [Feature #12573] Introduce a straightforward way to discover whether a process is running (shyouhei)
  • [Feature #13263] Add companion integer nth-root method to recent Integer#isqrt (shyouhei)
  • [Feature #9453] Return symbols of defined methods for attr and friends (shyouhei) look at the last comment
  • [Feature #13302] Provide a (force) –enable-openssl switch for ruby ./configure (or similar) (shyouhei)
  • [Feature #13303] String#any? as !String#empty? (shyouhei)

From attendees

  • [Feature #13334] Removed deprecated mathn extentions. (hsbt)
  • Assign who? section (shyouhei)
    • [Bug #13228] s[i]=c(assigning a character) for String is slower than Array on Linux
    • [Bug #13264] Binding#irb does not work in context of frozen object
    • [Bug #12642] Net::HTTP populates host header incorrectly when using an IPv6 Address
    • [Bug #13196] Improve keyword argument errors when non-keyword arguments given
    • [Bug #13320] rescue blocks get an entry in backtrace locations
    • [Bug #13324] IRB Segmentation Fault from eval infinite loop
    • [Bug #13330] Array.include? is slow for symbols
    • [Bug #13336] Default Parameters don’t work
    • [Bug #13337] Eval and Later Defined Local Variables
    • [Bug #13338] MinGW SEGV in test/ruby/test_keyword.rb svn 58034, ok in svn 58021.
    • [Bug #13390] MinGW build test-all SEGV, issue in test framework or error recovery?
    • [Bug #13350] File.read :newline option not respected on Linux
    • [Feature #13362] [PATCH] socket: avoid fcntl for read/write_nonblock on Linux
    • [Bug #13351] net/http: Net::HTTP.start sets wrong default arg value
    • watson’s
      • [Bug #13341] Improve performance of implicit type conversion
      • [Bug #13342] Improve yielding block performance
      • [Bug #13354] Improve Time#<=> performance
      • [Bug #13357] Improve Time#+ & Time#- performance
      • [Bug #13365] Improve performance of rb_equal() with special constants
      • [Bug #13368] Improve performance of Array#sum with float elements
      • [Bug #13374] Fix one of performance regressions in method calling
    • [Bug #13373] FileUtils methods for copy, move and remove directories is not providing a decent error trace for letting know if it was success or fail
    • [Bug #13101] Date#rfc2822 and Time#rfc2822 don’t return the same format
    • [Bug #13312] String#casecmp raises TypeError instead of returning nil
    • [Bug #12684] Delegator#eql? missing
    • [Feature #13389] [PATCH] POP3 support timeout for TLS handshake
    • [Bug #13319] GC issues seen with GCC7
    • [Bug #13404] Hash#any? yields arguments to lambdas with proc semantics
    • [Bug #13406] URI.parse
    • [Bug #13413] –with-static-linked-ext doesn’t install extension files on make install
    • [Bug #13420] Integer#{round,floor,ceil,truncate} should always return an integer, not a float
    • [Bug #13429] Net::SMTP has no read timeout when connexion over TLS
  • [Feature #13309] Locale paramter for Integer(), Float(), Rational() (shyouhei)
  • [Feature #12733] Bundle bundler to ruby core (shyouhei) updates?
  • [Feature #11302] Dir.entries and Dir.foreach without [”.”, “..”] (shyouhei)
  • [Feature #13332] Forwardable#def_instance_delegator nil (shyouhei)
  • [Feature #13334] Removed deprecated mathn extentions. (shyouhei)
  • [Bug #12921] Retrieve user and password for proxy from env (shyouhei)
  • [Feature #13333] block to yield (shyouhei)
  • [Feature #13378] Eliminate 4 of 8 syscalls when requiring file by absolute path (shyouhei)
  • [Feature #13381] [PATCH] Expose rb_fstring and its family to C extensions (shyouhei)
  • [Bug #13315] Single “%” at the end of printf format string appears in the result (shyouhei)
  • [Feature #13385] [PATCH] Make Resolv::DNS::Name validation similar to host and dig commands (shyouhei)
  • [Feature #13383] [PATCH] Module#source_location (shyouhei)
  • [Feature #12063] KeyError#receiver and KeyError#name (shyouhei) status?
  • [Bug #13397] #object_id should not be signed (shyouhei)
  • [Feature #13396] Net::HTTP has no write timeout (shyouhei)
  • [Bug #13407] We have recv_nonblock but not send_nonblock… can we add it? (shyouhei)
  • [Feature #13395] Add a method to check for not nil (shyouhei)

From non-attendees

Write your name and your interest (what do you want to ask and to whom?) please.

  • example: [Feature #10917] Add GC.stat[:total_time] when GC profiling enabled (ko1)

  • [Feature #12886] URI#merge doesn’t handle paths correctly (hsbt)

Log

Date: 2017/04/17 (Mon)

Time: 14:00- 19:00 (JST)

Place: Cookpad Inc. Headquarter

Sign-up: https://ruby.connpass.com/event/53124/

log edit: https://docs.google.com/document/d/1_tWfg7_t-JRXgGlMaOGnpyipGP7P-lKQ18NtciWhKIY/edit

log: TBD

Next meeting

  • 5/19

  • at speee, roppongi

About 2.5 timeframe

https://bugs.ruby-lang.org/projects/ruby-trunk/wiki/ReleaseEngineering25

  • Previrew 1 in June?

  • 合宿

  • Nov-Dec?
  • Shimane? or Kanto?

Current status of Ruby 2.4

  • 2.4.1 released.
  • Chikanaga-san is handling.

Carry-over from previous meeting(s)

  • [Bug #13257] Symbol#respond_to?(:singleton_class) should be false (naruse)

  • akr: I feel it’s OK to undef them.
  • matz: I doubt to check behaviours using respond_to?
  • nalsh: class « obj; creates a singleton class if not.  This is not good.

  • nobu: It seems what OP wants is actually a list of extended modules, which is not currently obtainable.

  • [Misc #13163] Uncaught exceptions may not be reported when Thread#report_on_exception=true and Thread#abort_on_exception=true (naruse)

  • nobu: OK to me.

  • [Bug #13181] Unexpected line in rescue backtrace (shyouhei) what’s this?

  • assign ko1

  • [Feature #8263] Support discovering yield state of individual Fibers (shyouhei)

  • assign ko1

  • [Feature #8576] Add optimized method type for constant value methods (shyouhei)

  • assign ko1

  • [Bug #13188] Reinitialize Ruby VM. (shyouhei) New C API …?

  • assign ko1

  • [Feature #2740] Extend const_missing to pass in the nesting (shyouhei)

  • matz: does this still work?

  • matz: also, I don’t like autoloading in general.

  • [Feature #13211] Hash#delete taking a splat (shyouhei)

  • mrkn: I once wanted this too
  • akr: return value? array for arity >= 2?
  • nobu: I wonder if this is already there in ActiveSupport
  • mrkn: That’s Hash#except!
  • shyouhei: seems the OP wants to split a hash into two.
  • mrkn: That’s Hash#extract!

  • akr: seems what is requested is not a good design, compared to what is wanted.

  • [Bug #11567] Segmentation fault CFUNC :gets (shyouhei)

  • nobu: it seems race between read and close

  • shyouhei: seems fixed.

  • [Feature #13172] Method that yields object to block and returns result (shyouhei) new name candidate

  • shyouhei: #pass is proposed.

  • matz: It doesn’t sound nice.

  • [Feature #13224] Add FrozenError as a subclass of RuntimeError (shyouhei)

  • matz: I don’t want to have infinite number of exceptions but this seems OK.

  • [Misc #13230] Better Do … while structure (shyouhei)

  • shyouhei: I don’t understand this.

  • matz: reject

  • [Feature #13245] [PATCH] reject inter-thread TLS modification (shyouhei)

  • ko1: we would like to forbid this in a future.
  • shyouhei: what about adding warning.

  • akr: is this need?  We have no problem right now.  Is this worth introducing backwards incompatibility?

  • [Feature #13252] C API for creating strings without copying (shyouhei)

  • ko1: the proposal is not doable
  • nobu: maybe we need to have separate deallocator

  • assign ko1

  • [Bug #13249] Access modifiers don’t have an effect inside class methods in Ruby >= 2.3 (shyouhei) design issue or bug?

  • naruse: wasn’t this intentional?
  • ko1: no idea.

  • matz: It’s OK to me to deprecate this, if we can warn this.

  • [Feature #13265] TracePoint for basic operation redefinition (shyouhei)

  • assign ko1

  • [Bug #12834] prepend getting prepended even if it already exists in the ancestors chain (shyouhei)

  • akr: doesn’t this loop forever?
  • ko1: no. I fixed that loop.
  • matz: I wanted to change include’s behaviour in 1.9 but not successful then.

  • matz: I started thinking prepend shall be applied many times. For include..?

  • [Bug #7976] TracePoint call is at call point, not call site (shyouhei)

  • ko1: わかる

  • assign ko1

  • [Feature #12901] Anonymous functions without scope lookup overhead (shyouhei) status?

  • pass this time.

  • [Feature #12573] Introduce a straightforward way to discover whether a process is running (shyouhei)

  • akr: locked pidfile does not need this method.

  • [Feature #13263] Add companion integer nth-root method to recent Integer#isqrt (shyouhei)

  • shyouhei: I have no idea when this is used.
  • akr: same as above.

  • mrkn: what about asking the OP?

  • [Feature #9453] Return symbols of defined methods for attr and friends (shyouhei) look at the last comment

  • shyouhei: the last comment says the situation changed.
  • akr: that sounds reasonable…
  • matz: but definition of multiple methods at once is something special here.

  • matz: will respond.

  • [Feature #13302] Provide a (force) –enable-openssl switch for ruby ./configure (or similar) (shyouhei)

  • shyouhei: openssl needs miniruby and doesn’t stop if absent.
  • naruse: –enable-openssl sounds strange, it should be –with-openssl

  • nobu: what about let –with-ext=openssl to abort if absent.

  • [Feature #13303] String#any? as !String#empty? (shyouhei)

  • matz: I don’t like any? because Enumerable#any? cannot be applied to String straight-forward.
  • matz: str.not_empty? seems inferior than !str.empty?
  • naruse: !str.empty? does not work for nil.  I want to do str_or_nil&.not_empty?

  • name candidates:

  • NG: present
  • NG: empty?
  • candicate: non_empty?
  • candicate: not_empty?

From attendees

  • [Feature #13334] Removed deprecated mathn extentions. (hsbt)

  • matz: OK.

  • Assign who? section (shyouhei)

  • [Bug #13228] s[i]=c(assigning a character) for String is slower than Array on Linux

  • shyouhei: that search_nonascii is unavoidable
  • naruse: 5x slower on my machine
  • nobu: no difference on my macine

  • shyouhei: what’s the cause?

  • [Bug #13264] Binding#irb does not work in context of frozen object
  • [Bug #12642] Net::HTTP populates host header incorrectly when using an IPv6 Address
  • [Bug #13196] Improve keyword argument errors when non-keyword arguments given
  • [Bug #13320] rescue blocks get an entry in backtrace locations
  • [Bug #13324] IRB Segmentation Fault from eval infinite loop
  • [Bug #13330] Array.include? is slow for symbols
  • [Bug #13336] Default Parameters don’t work
  • [Bug #13337] Eval and Later Defined Local Variables
  • [Bug #13338] MinGW SEGV in test/ruby/test_keyword.rb svn 58034, ok in svn 58021.
  • [Bug #13390] MinGW build test-all SEGV, issue in test framework or error recovery?
  • [Bug #13350] File.read :newline option not respected on Linux
  • [Feature #13362] [PATCH] socket: avoid fcntl for read/write_nonblock on Linux
  • [Bug #13351] net/http: Net::HTTP.start sets wrong default arg value

  • watson’s

  • [Bug #13341] Improve performance of implicit type conversion
  • [Bug #13342] Improve yielding block performance
  • [Bug #13354] Improve Time#<=> performance
  • [Bug #13357] Improve Time#+ & Time#- performance
  • [Bug #13365] Improve performance of rb_equal() with special constants
  • [Bug #13368] Improve performance of Array#sum with float elements
  • [Bug #13374] Fix one of performance regressions in method calling

  • nobu: I call him for a volunteer.

  • [Bug #13373] FileUtils methods for copy, move and remove directories is not providing a decent error trace for letting know if it was success or fail
  • [Bug #13101] Date#rfc2822 and Time#rfc2822 don’t return the same format
  • [Bug #13312] String#casecmp raises TypeError instead of returning nil
  • [Bug #12684] Delegator#eql? missing
  • [Feature #13389] [PATCH] POP3 support timeout for TLS handshake
  • [Bug #13319] GC issues seen with GCC7
  • [Bug #13404] Hash#any? yields arguments to lambdas with proc semantics
  • [Bug #13406] URI.parse
  • [Bug #13413] –with-static-linked-ext doesn’t install extension files on make install
  • [Bug #13420] Integer#{round,floor,ceil,truncate} should always return an integer, not a float

  • [Bug #13429] Net::SMTP has no read timeout when connexion over TLS

  • [Feature #13309] Locale paramter for Integer(), Float(), Rational() (shyouhei)
  • [Feature #12733] Bundle bundler to ruby core (shyouhei) updates?
  • [Feature #11302] Dir.entries and Dir.foreach without ”.”, “..”
  • [Feature #13332] Forwardable#def_instance_delegator nil (shyouhei)
  • [Feature #13334] Removed deprecated mathn extentions. (shyouhei)

  • [Bug #12921] Retrieve user and password for proxy from env (shyouhei)

  • akr: what about whitelisting OSes? modern ones should work.

  • [Feature #13333] block to yield (shyouhei)
  • [Feature #13378] Eliminate 4 of 8 syscalls when requiring file by absolute path (shyouhei)
  • [Feature #13381] [PATCH] Expose rb_fstring and its family to C extensions (shyouhei)
  • [Bug #13315] Single “%” at the end of printf format string appears in the result (shyouhei)
  • [Feature #13385] [PATCH] Make Resolv::DNS::Name validation similar to host and dig commands (shyouhei)
  • [Feature #13383] [PATCH] Module#source_location (shyouhei)
  • [Feature #12063] KeyError#receiver and KeyError#name (shyouhei) status?
  • [Bug #13397] #object_id should not be signed (shyouhei)
  • [Feature #13396] Net::HTTP has no write timeout (shyouhei)
  • [Bug #13407] We have recv_nonblock but not send_nonblock… can we add it? (shyouhei)
  • [Feature #13395] Add a method to check for not nil (shyouhei)

From non-attendees

  • [Feature #12886] URI#merge doesn’t handle paths correctly (hsbt)

  • hsbt: cf https://github.com/ruby/ruby/pull/1469
  • hsbt: the fix seems partial. status?
  • nobu: the fix did not change behaviours.
  • hsbt: what about this issue then?
  • akr: martin’s comment #3 seems reasonable.