Project

General

Profile

Bug #7484

Fiddle::Importer incompatibility with DL::Importer

Added by ngoto (Naohisa Goto) almost 5 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Normal
Target version:
-
[ruby-core:50405]

Description

As described in r38054, Fiddle::Importer is not compatible with DL::Importer about :stdcall.
I hope attached patch fixed the issue.

stdcall.patch (615 Bytes) stdcall.patch ngoto (Naohisa Goto), 12/01/2012 02:39 AM

Associated revisions

Revision 38182
Added by ngoto (Naohisa Goto) almost 5 years ago

  • ext/fiddle/lib/fiddle/import.rb (import_function, bind_function): should respect call_type for migration from DL to Fiddle. [Bug #7484]

Revision 38182
Added by ngoto (Naohisa Goto) almost 5 years ago

  • ext/fiddle/lib/fiddle/import.rb (import_function, bind_function): should respect call_type for migration from DL to Fiddle. [Bug #7484]

Revision 38182
Added by ngoto (Naohisa Goto) almost 5 years ago

  • ext/fiddle/lib/fiddle/import.rb (import_function, bind_function): should respect call_type for migration from DL to Fiddle. [Bug #7484]

Revision 38182
Added by ngoto (Naohisa Goto) almost 5 years ago

  • ext/fiddle/lib/fiddle/import.rb (import_function, bind_function): should respect call_type for migration from DL to Fiddle. [Bug #7484]

History

#1 [ruby-core:50547] Updated by usa (Usaku NAKAMURA) almost 5 years ago

  • Status changed from Open to Assigned
  • Priority changed from Normal to 5

I'm waiting this problem is fixed.
I'm planning to move ext/dl/win32/lib/win32/* scripts to fiddle, but this problem prevents it.

#2 [ruby-core:50548] Updated by usa (Usaku NAKAMURA) almost 5 years ago

I asked aaron about this commit, and he confirmed the patch.
Goto-san, please commit it!

#3 Updated by ngoto (Naohisa Goto) almost 5 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r38182.
Naohisa, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • ext/fiddle/lib/fiddle/import.rb (import_function, bind_function): should respect call_type for migration from DL to Fiddle. [Bug #7484]

Also available in: Atom PDF