#9 Typo in FTLayoutGlue ftglGetLayoutBBox()

closed-accepted
None
5
2010-05-06
2008-09-21
No

The C-function ftglGetLayoutBBox() is not exported as it is misspelled in FTLayoutGlue.cpp.

In FTLayout.h it is spelled
ftgGetLayoutBBox()
but in FTLayoutGlue.cpp it is
ftgGetlLayoutBBox()

Note the "lL".

Discussion

  • Comment has been marked as spam. 
    Undo

    You can see all pending comments posted by this user  here

    Anonymous - 2009-11-19

    Fix:

    --- ftgl-2.1.3~rc5/src/FTLayout/FTLayoutGlue.cpp 2008-06-09 08:52:51.000000000 -0400
    +++ ftgl-2.1.3~rc5/src/FTLayout/FTLayoutGlue.cpp 2009-11-19 03:19:57.993721652 -0500
    @@ -76,13 +76,13 @@

    // virtual FTBBox FTLayout::BBox(const char* string)
    extern "C++" {
    -C_FUN(static FTBBox, _ftgGetlLayoutBBox, (FTGLlayout *l, const char *s),
    +C_FUN(static FTBBox, _ftglGetLayoutBBox, (FTGLlayout *l, const char *s),
    return static_ftbbox, BBox, (s));
    }

    -void ftgGetlLayoutBBox(FTGLlayout *l, const char * s, float c[6])
    +void ftglGetLayoutBBox(FTGLlayout *l, const char * s, float c[6])
    {
    - FTBBox ret = _ftgGetlLayoutBBox(l, s);
    + FTBBox ret = _ftglGetLayoutBBox(l, s);
    FTPoint lower = ret.Lower(), upper = ret.Upper();
    c[0] = lower.Xf(); c[1] = lower.Yf(); c[2] = lower.Zf();
    c[3] = upper.Xf(); c[4] = upper.Yf(); c[5] = upper.Zf();

     
  • Sean Morrison

    Sean Morrison - 2010-05-06
    • assigned_to: nobody --> brlcad
    • status: open --> open-accepted
     
  • Sean Morrison

    Sean Morrison - 2010-05-06

    Moving from bug tracker to patches tracker given it provides a patch.

     
  • Sean Morrison

    Sean Morrison - 2010-05-06
    • status: open-accepted --> closed-accepted
     
  • Sean Morrison

    Sean Morrison - 2010-05-06

    This patch has been applied as r1226 and should be included in the next FTGL release (which does not yet have a firm date, but should be 'soon' within the next month or so). Thanks!

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks