diff --git a/cocoa/font.m b/cocoa/font.m index f2f38793c..795b04802 100644 --- a/cocoa/font.m +++ b/cocoa/font.m @@ -149,7 +149,7 @@ static inline NSUInteger cocoa_glyph_for_location( NSLayoutManager *layout, CGFl NSUInteger glyphIndex = [layout glyphIndexForPoint: NSMakePoint( cocoa_px_to_pt( x ), 0 ) inTextContainer: cocoa_text_container fractionOfDistanceThroughGlyph: &fraction]; - if (fraction > 0) ++glyphIndex; + if (fraction >= 1.0) ++glyphIndex; return glyphIndex; } @@ -171,9 +171,11 @@ static NSLayoutManager *cocoa_prepare_layout_manager( const char *bytes, size_t const plot_font_style_t *style ) { if (NULL == bytes || 0 == length) return nil; - + + NSString *string = [[[NSString alloc] initWithBytes: bytes length:length encoding:NSUTF8StringEncoding] autorelease]; + if (string == nil) return nil; + static NSLayoutManager *layout = nil; - if (nil == layout) { cocoa_text_container = [[NSTextContainer alloc] initWithContainerSize: NSMakeSize( CGFLOAT_MAX, CGFLOAT_MAX )]; [cocoa_text_container setLineFragmentPadding: 0]; @@ -182,9 +184,6 @@ static NSLayoutManager *cocoa_prepare_layout_manager( const char *bytes, size_t [layout addTextContainer: cocoa_text_container]; } - - NSString *string = [[[NSString alloc] initWithBytes: bytes length:length encoding:NSUTF8StringEncoding] autorelease]; - static NSString *oldString = 0; static plot_font_style_t oldStyle = { 0 }; @@ -203,14 +202,13 @@ static NSLayoutManager *cocoa_prepare_layout_manager( const char *bytes, size_t [attributes release]; attributes = [cocoa_font_attributes( style ) retain]; } - + [cocoa_text_storage release]; cocoa_text_storage = [[NSTextStorage alloc] initWithString: string attributes: attributes]; [cocoa_text_storage addLayoutManager: layout]; [layout ensureLayoutForTextContainer: cocoa_text_container]; - return layout; }