So It seems to me the only change there is uninlining the lambda and assigning it to a throwaway variable. That feels like busywork. I suspect this has to do with keeping a reference so it doesn't get GC'd. Just a hunch. Am I getting there? Why is this needed? On 2011-01-09, at 17:10 , Matt Aimonetti wrote:

I realize I didn't really reply to your question, the following code should work:

{{{ def loadSprite(sender) handler = lambda do |code| return if code == NSCancelButton # do things with the panel's date otherwise end

@panel.beginSheetModalForWindow(@window, completionHandler: handler) end }}}

- Matt

On Sun, Jan 9, 2011 at 8:09 AM, Rolando Abarca <rolando@gamesforfood.com>wrote:

...

Hi all,

I've taken a look at the referred issue (#712), but I'm not sure of the final implemented syntax. I'm trying to use a NSOpenPanel, so far this is what I'm doing:

{{{ def loadSprite(sender) @panel.beginSheetModalForWindow(@window, completionHandler:lambda { |code| NSLog("code: #{code}") }) end }}}

what's the Right Way(tm) to use a NSOpenPanel? thanks! -- Rolando Abarca M. Games For Food S.p.A. http://www.gamesforfood.com Phone: +1 (408) 345-5433 _______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

_______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

This hack isn't needed anymore as of 0.8. Passing the Proc inline should just be fine. Laurent On Jan 9, 2011, at 12:22 PM, Matt Aimonetti wrote:

I suspect this has to do with keeping a reference so it doesn't get GC'd. Just a hunch. Am I getting there?

That's my guess too, if you pass the lambda as a param, it might get GC'd and when the panel is done, it won't have a reference to the handler.

- Matt

On Sun, Jan 9, 2011 at 11:46 AM, Caio Chassot <lists@caiochassot.com> wrote: So It seems to me the only change there is uninlining the lambda and assigning it to a throwaway variable. That feels like busywork.

I suspect this has to do with keeping a reference so it doesn't get GC'd. Just a hunch. Am I getting there?

Why is this needed?

On 2011-01-09, at 17:10 , Matt Aimonetti wrote:

...

I realize I didn't really reply to your question, the following code should work:

{{{ def loadSprite(sender) handler = lambda do |code| return if code == NSCancelButton # do things with the panel's date otherwise end

@panel.beginSheetModalForWindow(@window, completionHandler: handler) end }}}

- Matt

On Sun, Jan 9, 2011 at 8:09 AM, Rolando Abarca <rolando@gamesforfood.com>wrote:

...

Hi all,

I've taken a look at the referred issue (#712), but I'm not sure of the final implemented syntax. I'm trying to use a NSOpenPanel, so far this is what I'm doing:

{{{ def loadSprite(sender) @panel.beginSheetModalForWindow(@window, completionHandler:lambda { |code| NSLog("code: #{code}") }) end }}}

what's the Right Way(tm) to use a NSOpenPanel? thanks! -- Rolando Abarca M. Games For Food S.p.A. http://www.gamesforfood.com Phone: +1 (408) 345-5433 _______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

_______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

_______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

_______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

Laurent you are right, however I noticed a little gotcha: If you use the following code: panel.beginSheetModalForWindow window, completionHandler: Proc.new do |result| return if (result == NSCancelButton) path = panel.filename dest_path = applicationSupportFolder guid = NSProcessInfo.processInfo.globallyUniqueString dest_path = "#{dest_path}/#{guid}" error = Pointer.new(:id) NSFileManager.defaultManager.copyItemAtPath(path, toPath:dest_path, error:error) NSApp.presentError(error) if error[0] movie.setValue(dest_path, forKey:"imagePath") end You get an argument error: tried to create Proc object without a block (ArgumentError) However using the curly brackets would work: panel.beginSheetModalForWindow window, completionHandler: Proc.new{|result| return if (result == NSCancelButton) path = panel.filename dest_path = applicationSupportFolder guid = NSProcessInfo.processInfo.globallyUniqueString dest_path = "#{dest_path}/#{guid}" error = Pointer.new(:id) NSFileManager.defaultManager.copyItemAtPath(path, toPath:dest_path, error:error) NSApp.presentError(error) if error[0] movie.setValue(dest_path, forKey:"imagePath") } Finally, the only way to get the do/end proc approach to work is to use the parenthesis around the arguments: panel.beginSheetModalForWindow(window, completionHandler: Proc.new do |result| return if (result == NSCancelButton) path = panel.filename dest_path = applicationSupportFolder guid = NSProcessInfo.processInfo.globallyUniqueString dest_path = "#{dest_path}/#{guid}" error = Pointer.new(:id) NSFileManager.defaultManager.copyItemAtPath(path, toPath:dest_path, error:error) NSApp.presentError(error) if error[0] movie.setValue(dest_path, forKey:"imagePath") end) - Matt On Sun, Jan 9, 2011 at 2:22 PM, Laurent Sansonetti <lsansonetti@apple.com>wrote:

Hi Rolando,

The syntax is simple, you simply pass a Proc project.

Here is an example:

framework 'Foundation' a = [1, 2, 3, 4, 5] a.enumerateObjectsUsingBlock(Proc.new { |obj, index, stop| p obj stop.assign(true) if index == 2 })

It looks like the snippet you pasted should just work. Did you find a problem?

Laurent

On Jan 9, 2011, at 8:09 AM, Rolando Abarca wrote:

Hi all,

I've taken a look at the referred issue (#712), but I'm not sure of the final implemented syntax. I'm trying to use a NSOpenPanel, so far this is what I'm doing:

{{{ def loadSprite(sender) @panel.beginSheetModalForWindow(@window, completionHandler:lambda { |code| NSLog("code: #{code}") }) end }}}

what's the Right Way(tm) to use a NSOpenPanel? thanks! -- Rolando Abarca M. Games For Food S.p.A. http://www.gamesforfood.com Phone: +1 (408) 345-5433 _______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

_______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

You are right, still a gotcha tho: Ruby 1.9.2: def foo(a, b={}) b[:handler].call if b.has_key?(:handler) end foo :bar, handler: Proc.new do puts 42 end `new': tried to create Proc object without a block (ArgumentError) def foo(a, b={}) b[:handler].call if b.has_key?(:handler) end foo :bar, handler: Proc.new{ puts 42 } Works fine. I guess this is due to the fact that we are using CRuby's parser. I got a bit confused because MacRuby allows you to define methods like that: def foo(a, b:handler) handler.call end foo :bar, b: Proc.new{ puts 42 } Which is an extension of the Ruby syntax. Anyways, thanks for the heads up, I didn't notice it got fixed in 0.8 - Matt On Sun, Jan 9, 2011 at 2:44 PM, Laurent Sansonetti <lsansonetti@apple.com>wrote:

This seems to be a Ruby syntax point, unrelated to MacRuby.

Laurent

On Jan 9, 2011, at 2:32 PM, Matt Aimonetti wrote:

Laurent you are right, however I noticed a little gotcha:

If you use the following code:

panel.beginSheetModalForWindow window, completionHandler: Proc.new do |result| return if (result == NSCancelButton) path = panel.filename dest_path = applicationSupportFolder guid = NSProcessInfo.processInfo.globallyUniqueString dest_path = "#{dest_path}/#{guid}" error = Pointer.new(:id) NSFileManager.defaultManager.copyItemAtPath(path, toPath:dest_path, error:error) NSApp.presentError(error) if error[0] movie.setValue(dest_path, forKey:"imagePath") end

You get an argument error: tried to create Proc object without a block (ArgumentError)

However using the curly brackets would work:

panel.beginSheetModalForWindow window, completionHandler: Proc.new{|result| return if (result == NSCancelButton) path = panel.filename dest_path = applicationSupportFolder guid = NSProcessInfo.processInfo.globallyUniqueString dest_path = "#{dest_path}/#{guid}" error = Pointer.new(:id) NSFileManager.defaultManager.copyItemAtPath(path, toPath:dest_path, error:error) NSApp.presentError(error) if error[0] movie.setValue(dest_path, forKey:"imagePath") }

Finally, the only way to get the do/end proc approach to work is to use the parenthesis around the arguments:

panel.beginSheetModalForWindow(window, completionHandler: Proc.new do |result| return if (result == NSCancelButton) path = panel.filename dest_path = applicationSupportFolder guid = NSProcessInfo.processInfo.globallyUniqueString dest_path = "#{dest_path}/#{guid}" error = Pointer.new(:id) NSFileManager.defaultManager.copyItemAtPath(path, toPath:dest_path, error:error) NSApp.presentError(error) if error[0] movie.setValue(dest_path, forKey:"imagePath") end)

- Matt

On Sun, Jan 9, 2011 at 2:22 PM, Laurent Sansonetti <lsansonetti@apple.com>wrote:

...

Hi Rolando,

The syntax is simple, you simply pass a Proc project.

Here is an example:

framework 'Foundation' a = [1, 2, 3, 4, 5] a.enumerateObjectsUsingBlock(Proc.new { |obj, index, stop| p obj stop.assign(true) if index == 2 })

It looks like the snippet you pasted should just work. Did you find a problem?

Laurent

On Jan 9, 2011, at 8:09 AM, Rolando Abarca wrote:

Hi all,

I've taken a look at the referred issue (#712), but I'm not sure of the final implemented syntax. I'm trying to use a NSOpenPanel, so far this is what I'm doing:

{{{ def loadSprite(sender) @panel.beginSheetModalForWindow(@window, completionHandler:lambda { |code| NSLog("code: #{code}") }) end }}}

what's the Right Way(tm) to use a NSOpenPanel? thanks! -- Rolando Abarca M. Games For Food S.p.A. http://www.gamesforfood.com Phone: +1 (408) 345-5433 _______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

_______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

_______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

_______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

Yep, Chad Fowler explained to me that it's the difference between precedence and binding with do/end vs {} - Matt On Sun, Jan 9, 2011 at 5:57 PM, Vincent Isambart <vincent.isambart@gmail.com

wrote:

...

You are right, still a gotcha tho:

Ruby 1.9.2:

def foo(a, b={}) b[:handler].call if b.has_key?(:handler) end foo :bar, handler: Proc.new do puts 42 end

`new': tried to create Proc object without a block (ArgumentError)

Works if you add parentheses though : def foo(a, b={}) b[:handler].call if b.has_key?(:handler) end foo :bar, handler: (Proc.new do puts 42 end) _______________________________________________ MacRuby-devel mailing list MacRuby-devel@lists.macosforge.org http://lists.macosforge.org/mailman/listinfo.cgi/macruby-devel

On Jan 11, 2011, at 5:40 PM, Rolando Abarca wrote:

I'm using MacRuby 0.8

Works for me! macruby -version MacRuby 0.9 (ruby 1.9.2) [universal-darwin10.0, x86_64] macirb irb(main):001:0> framework 'Foundation' => true irb(main):002:0> a = [1,2,3,4,5] => [1, 2, 3, 4, 5] irb(main):003:0> a.enumerateObjectsUsingBlock(Proc.new { |obj, index, stop| irb(main):004:1> p obj irb(main):005:1> stop.assign(true) if index == 2 irb(main):006:1> }) 1 2 3 => [1, 2, 3, 4, 5] --- http://wagerlabs.com | @wagerlabs | http://www.linkedin.com/in/joelreymont

Matt, I -am- using the latest BridgeSupport (Preview 2) if you are talking to me. On Jan 11, 2011, at 6:06 PM, Matt Aimonetti wrote:

Can you make sure you are using the latest BridgeSupport?

- Matt

Sent from my iPhone

On Jan 11, 2011, at 10:01, Joel Reymont <joelr1@gmail.com> wrote:

...

On Jan 11, 2011, at 5:40 PM, Rolando Abarca wrote:

...

I'm using MacRuby 0.8

Works for me!

macruby -version MacRuby 0.9 (ruby 1.9.2) [universal-darwin10.0, x86_64]

macirb irb(main):001:0> framework 'Foundation' => true irb(main):002:0> a = [1,2,3,4,5] => [1, 2, 3, 4, 5] irb(main):003:0> a.enumerateObjectsUsingBlock(Proc.new { |obj, index, stop| irb(main):004:1> p obj irb(main):005:1> stop.assign(true) if index == 2 irb(main):006:1> }) 1 2 3 => [1, 2, 3, 4, 5]

--- http://wagerlabs.com | @wagerlabs | http://www.linkedin.com/in/joelreymont

You're right: using BridgeSupport Preview 2 works! :-) Thanks! -- Rolando Abarca M. Games For Food S.p.A. http://www.gamesforfood.com Phone: +1 (408) 345-5433