This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [gui][PATCH] Add missing Font fields


On Tue, 2004-11-02 at 17:22, Tom Tromey wrote:
> >>>>> "Tom" == Thomas Fitzsimmons <fitzsim@redhat.com> writes:
> 
> Tom> 2004-11-02  Thomas Fitzsimmons  <fitzsim@redhat.com>
> Tom> 	* java/awt/Font.java (name): New field.
> Tom> 	(size): Likewise.
> Tom> 	(style): Likewise.
> 
> This seems a little weird to me, since the new fields aren't actually
> used anywhere.  Are they supposed to mean something to subclasses?  I
> understand wanting to have them to claim API compatibility

Yeah, this is the reason I added them.

>  -- but in
> this form aren't they sort of like writing stubbed methods?
> 

Yes, I guess I was being lazy -- I'll commit a patch that sets these
values in the Font constructors.  I'm not sure how programs use the
fields though.

Tom



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]