JRuby

probably extraneous warning message generated

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: JRuby 1.7.0.pre2
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

this snippet:

require 'java'

class JFrameDescendant < javax.swing.JFrame

  def initialize
     @panel = 3
  end
end
JFrameDescendant.new

Results in this message:

repro.rb:1 warning: instance vars on non-persistent Java type
JFrameDescendant (http://wiki.jruby.org/Persistence)

Which may not be accurate, since the descendant is pure ruby, so should be happy with instance vars, shouldn't it?

see http://www.ruby-forum.com/topic/4341042

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Charles Oliver Nutter added a comment -

Yeah, this is extraneous. The link between the concrete Java class and the Ruby class is hard and it should be possible to safely associate instance variables in that scenario. I'll poke around.

Show
Charles Oliver Nutter added a comment - Yeah, this is extraneous. The link between the concrete Java class and the Ruby class is hard and it should be possible to safely associate instance variables in that scenario. I'll poke around.
Hide
Permalink
Charles Oliver Nutter added a comment - 
commit 066c2f03a051213276866cb441e2e76cd43cc397
Author: Charles Oliver Nutter <headius@headius.com>
Date:   Fri Jun 29 11:13:01 2012 -0500

    Fix JRUBY-6721
    
    probably extraneous warning message generated
    
    Mark Java subclasses as "cache proxy" to silence warnings.
Show
Charles Oliver Nutter added a comment - commit 066c2f03a051213276866cb441e2e76cd43cc397 Author: Charles Oliver Nutter <headius@headius.com> Date: Fri Jun 29 11:13:01 2012 -0500 Fix JRUBY-6721 probably extraneous warning message generated Mark Java subclasses as "cache proxy" to silence warnings.
Hide
Permalink
Theo Hultberg added a comment -

I get this warning when I require 'ant' in 1.7.0.RC1, is it the same bug or another one?

~/.rvm/rubies/jruby-1.7.0.RC1/lib/ruby/shared/ant/element.rb:61 warning: instance vars on non-persistent Java type Java::OrgApacheToolsAnt::UnknownElement (http://wiki.jruby.org/Persistence)
Show
Theo Hultberg added a comment - I get this warning when I require 'ant' in 1.7.0.RC1, is it the same bug or another one? ~/.rvm/rubies/jruby-1.7.0.RC1/lib/ruby/shared/ant/element.rb:61 warning: instance vars on non-persistent Java type Java::OrgApacheToolsAnt::UnknownElement (http: //wiki.jruby.org/Persistence)
Hide
Permalink
Roger Pack added a comment -

what's at element.rb:61 ?

Show
Roger Pack added a comment - what's at element.rb:61 ?
Hide
Permalink
Theo Hultberg added a comment -

Looks like it's line 56 in current head: https://github.com/jruby/jruby/blob/master/lib/ruby/shared/ant/element.rb#L56

Show
Theo Hultberg added a comment - Looks like it's line 56 in current head: https://github.com/jruby/jruby/blob/master/lib/ruby/shared/ant/element.rb#L56
Hide
Permalink
Roger Pack added a comment -

Hmm I'm not sure it doesn't seem immediately obvious if it's adding an instance variable to a java class or not in there...

Show
Roger Pack added a comment - Hmm I'm not sure it doesn't seem immediately obvious if it's adding an instance variable to a java class or not in there...
Hide
Permalink
Theo Hultberg added a comment -

Here's a minimal command to reproduce the error:

ruby -e 'require "ant"; ant { echo :message => "hello" }'
Show
Theo Hultberg added a comment - Here's a minimal command to reproduce the error: ruby -e 'require "ant" ; ant { echo :message => "hello" }'
Hide
Permalink
Theo Hultberg added a comment -

The instance variable is created when the ant variable is accessed, because ant is an attr_accessor method (see https://github.com/jruby/jruby/blob/master/lib/ruby/shared/ant/element.rb#L9)

Can I reopen this bug somehow, or should I open a new one?

Show
Theo Hultberg added a comment - The instance variable is created when the ant variable is accessed, because ant is an attr_accessor method (see https://github.com/jruby/jruby/blob/master/lib/ruby/shared/ant/element.rb#L9 ) Can I reopen this bug somehow, or should I open a new one?
Hide
Permalink
Roger Pack added a comment -

I'd open a new one.

Show
Roger Pack added a comment - I'd open a new one.
Hide
Permalink
Theo Hultberg added a comment -

I opened JRUBY-6918

Show
Theo Hultberg added a comment - I opened JRUBY-6918

People

  • Assignee:
    Charles Oliver Nutter
    Reporter:
    Roger Pack
Vote (1)
Watch (4)

Dates

  • Created:
    Updated:
    Resolved: