#744: map on an array delivered by terminal app via scripting bridge gives an immutable error ----------------------------+----------------------------------------------- Reporter: jazzbox@… | Owner: lsansonetti@… Type: defect | Status: reopened Priority: major | Milestone: MacRuby 0.7 Component: MacRuby | Resolution: Keywords: | ----------------------------+----------------------------------------------- Changes (by jazzbox@…): * status: closed => reopened * resolution: invalid => Comment: Hi Thibault, I'm sorry, but I cannot agree! Your Obj-C example is a replacement for map! and not for map which should return a new Array and is not allowed to modify the original array! In the following example map works even with an immutable NSArray {{{ a = NSArray.arrayWithArray [1,2,3] p a p a.map { |e| e + 10 } p a # a should be unchanged }}} and gives correctly {{{ [1, 2, 3] [11, 12, 13] [1, 2, 3] }}} If I define my own Array#map my breaking example is working too, see this: {{{ framework 'ScriptingBridge' class NSArray # why does Array not work here? def my_map &block res = [] self.each { |e| res << block.call(e) } res end end term = SBApplication.applicationWithBundleIdentifier('com.apple.terminal') p term.windows p term.windows.class.ancestors p [term.windows.first].map { |w| w.name } p NSArray.arrayWithArray(term.windows).map { |w| w.name } p term.windows.my_map { |w| w.name } }}} And last but not least here is my working Obj-C example (although I don't like Obj-C very much, but hey, that's my first example with blocks) {{{ #import <Foundation/Foundation.h> #import <ScriptingBridge/ScriptingBridge.h> int main (int argc, const char * argv[]) { NSAutoreleasePool * pool = [[NSAutoreleasePool alloc] init]; SBApplication *app = [SBApplication applicationWithBundleIdentifier:@"com.apple.terminal"]; SBElementArray *windows = [app windows]; //NSLog(@"%@", [windows class]); NSLog(@"%@", windows); NSMutableArray *result = [NSMutableArray arrayWithCapacity: [windows count]]; [windows enumerateObjectsUsingBlock:^(id obj, NSUInteger idx, BOOL *stop) { //NSLog(@"%@", [obj name]); [result addObject:[obj name]]; }]; NSLog(@"%@", result); [pool drain]; return 0; } }}} BTW, the original example is still working with my version of MacRuby 0.5! -- Ticket URL: <http://www.macruby.org/trac/ticket/744#comment:2> MacRuby <http://macruby.org/>

#744: map on an array delivered by terminal app via scripting bridge gives an immutable error ----------------------------+----------------------------------------------- Reporter: jazzbox@… | Owner: lsansonetti@… Type: defect | Status: closed Priority: major | Milestone: MacRuby 0.7 Component: MacRuby | Resolution: fixed Keywords: | ----------------------------+----------------------------------------------- Changes (by martinlagardette@…): * status: reopened => closed * resolution: => fixed * milestone: => MacRuby 0.7 Comment: Ok, so the problem was actually how `#map` was implemented on `NSArray`s. It was using a `-mutableCopy` as the array to modify, but apparently this doesn't work with `SBElementArray`s. So I went and modified all the ruby methods on NSArray that would copy + modify an array (`#map`, `#shuffle`, `#uniq`, etc.) to use `[NSMutableArray arrayWithArray:rcv]` instead. This might be a little slower, but at least it works :-)/ For the curious, `[[rcv class] arrayWithArray:rcv]` could have been better, but this doesn't work with `SBEelementArray`s either. Anyway, fixed with r4253 :-) -- Ticket URL: <http://www.macruby.org/trac/ticket/744#comment:4> MacRuby <http://macruby.org/>