Menu

#47 Bugfix / enhancement

closed
zvtm (32)
5
2012-10-24
2008-06-17
greenfreak
No

I had a need to extend CGlyph.java and to create it with no primary or secondary glyphs (basically, a blank constructor) initially. Rather than adding extra code, I added this check. An alternative to accepting this patch is to create a blank constructor. Documentation should be added as well to remind the user to set the primary glyph if creating as a blank constructor (did not do).

Discussion

  • greenfreak

    greenfreak - 2008-06-17

    Patch for CGlyph

     
  • Emmanuel Pietriga

    Logged In: YES
    user_id=616074
    Originator: NO

    Patch applied in SVN on 20080617. Will be in release that follows 0.9.5. Thanks.

     

Anonymous
Anonymous

Add attachments
Cancel