Forum: Ruby-core Patch for : ext\dl\handle.c

Hi,

It's the first time I try to contribute to the ruby project.
I'd like to propose this small patch to make the dlhandle_sym function a
little more readable.
     * ext/dl/handle.c (dlhandle_sym): Added xdlsym function and 
replaced
CHECK_DLERROR with 'xdlsym'

Initially, I wanted to add the support for loading of symbols with the
format "_function@NNN" (from MSCV dlls) but the "dlhandle_sym" function 
was
a little to complex to implement it without some refactoring. I will 
need
some more patches to achieve this :)

I tried to keep this patch very simple.
What do you think?

Christian Bodt

Hi,

At Fri, 5 Mar 2010 22:24:01 +0900,
Christian Bodt wrote in [ruby-core:28504]:
> Initially, I wanted to add the support for loading of symbols with the
> format "_function@NNN" (from MSCV dlls) but the "dlhandle_sym" function was
> a little to complex to implement it without some refactoring. I will need
> some more patches to achieve this :)

It's already supported.