[OpenJDK 2D-Dev] Thoughts about font implementation

[Roman Kennke]

Hi Igor,

> Minimal changes are always good idea.
> However, we usually do not add new methods unless we actually use them.
> Not sure what's the best approach in this case.

Maybe we should give you the FontManager work first (it's not finished
yet, I have to test and fix on Windows first - this is aweful and takes
a lot of time to get going...), then I could also implement some more
alternative code paths on top this fonts-from-bytebuffer patch, as
suggested.

/Roman

> 
> -igor
> 
> Roman Kennke wrote:
> >> Am i understanding this correctly and this adds new TrueType constructor 
> >> that is not used in the OpenJDK itself?
> >>     
> >
> > For now, yes. (I'm using this in my own FontManager implementation in
> > the JamaicaVM, to load fonts from ROM instead of files. But this
> > requires the FontManager refactoring in Caciocavallo too...) I will most
> > likely add more alternative code paths on top of it later (for example,
> > for Font.createFont(InputStream) to not create a temporary file, or
> > generally for creating fonts using mmapped files instead of RAF,
> > possibly activated by a system property or so). I was thinking that it
> > would be easier to review if I send a minimal patch for the start.
> >
> > /Roman
> >
> >   
> >> -igor
> >>
> >> Roman Kennke wrote:
> >>     
> >>> Hi there,
> >>>
> >>> What do you think about this patch (see webrev link below)? Can we get
> >>> this into jdk7? It is very useful for me and I wouldn't like to maintain
> >>> it myself...
> >>>
> >>> Cheers, Roman
> >>>
> >>>   
> >>>       
> >>>> http://kennke.org/~roman/truetypefrombuffer/webrev/
> >>>>     
> >>>>         
> >>>   
> >>>       
> 
-- 
http://kennke.org/blog/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Dies ist ein digital signierter Nachrichtenteil
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20081015/3ba59abc/signature.asc>