TypeError thrown in DynamicMetaObjectBinder.Bind

murena · May 3, 2009, 12:14pm

Hi,
I’m reviewing ironruby-hpricot to clean up its code but I’ve just
noticed that something has changed in IronRuby since I last tested my
library. Please refer to the the following gist for a simplified test
case to reproduce the problem:

gist.github.com

https://gist.github.com/nrk/105928

repl_session.txt

IronRuby 0.4.0.0 on .NET 2.0.50727.4918
Copyright (c) Microsoft Corporation. All rights reserved.

>>> load_assembly "test"
=> true
>>> source = open("data.xml")
=> #<File:data.xml>
>>> Test::Nrk.read(source)
C:\Development\ironruby\working-nrk\ndp\fx\src\Core\Microsoft\Scripting\Actions\DynamicMetaObjectBinder.cs:87:in `Bind': The result type 'System.Object' of the binder 'read(C,0)' is not compatible with the result type 'IronRuby.Builtins.MutableString' expected by the call site. (TypeError)
        from C:\Development\ironruby\working-nrk\ndp\fx\src\Core\Microsoft\Scripting\Actions\C

This file has been truncated. show original

test.cs

using System;
using System.Runtime.CompilerServices;
using IronRuby.Builtins;
using IronRuby.Runtime;
using IronRuby.Runtime.Calls;
using Microsoft.Scripting;
using Microsoft.Scripting.Runtime;

namespace Test {
    using RubyIOReadCallSite = CallSite<Func<CallSite, RubyIO, MutableString>>;

This file has been truncated. show original

I’m pretty sure this was working ~1 month ago, am I doing something
wrong?

–
Daniele A.
http://www.clorophilla.net/blog/
http://twitter.com/JoL1hAHN

murena · May 3, 2009, 11:30pm

I think you need to do an explicit conversion to MutableString. The DLR
used to do some automatic conversions, but now you need to be explicit.
I don’t know what code to use so you will have to wait until someone who
knows that can reply…

murena · May 4, 2009, 7:39pm

A couple of issues:

As Shri said, the return type of a site must be Object.
You shouldn’t be creating a new site (and also not a shared one) in
each invocation of the method. Instead, use one of the CallSiteStorages
defined in CallSiteStorages.cs as the first parameter. You can search in
the built-ins for examples of usage.

Tomas

murena · May 4, 2009, 8:29pm

Hi Tomas,

Oh, I see. I didn’t know that, and the fact that it was working
with previous versions of IR just misleaded me.
Indeed, I know it was wrong to create a new site on each invocation
and that’s exactly why I’m reviewing the oldest bits of code that
actually interact with IronRuby to clean up everything and do it the
right way

Thanks!

On Mon, May 4, 2009 at 19:39, Tomas M.
[email protected] wrote:

Subject: Re: [Ironruby-core] TypeError thrown in DynamicMetaObjectBinder.Bind
I’m reviewing ironruby-hpricot to clean up its code but I’ve just
clorophilla.net

Ironruby-core mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/ironruby-core

–
Daniele A.
http://www.clorophilla.net/blog/
http://twitter.com/JoL1hAHN