@@ -394,7 +394,6 @@ impl ViewportRepaintInfo {
394394#[ derive( Default ) ]
395395struct ContextImpl {
396396 fonts : Option < FontStore > ,
397- font_definitions : FontDefinitions ,
398397
399398 memory : Memory ,
400399 animation_manager : AnimationManager ,
@@ -552,33 +551,29 @@ impl ContextImpl {
552551fn update_fonts_mut ( & mut self ) {
553552 profiling:: function_scope!( ) ;
554553 let input = & self . viewport ( ) . input ;
555- let pixels_per_point = input. pixels_per_point ( ) ;
556554 let max_texture_side = input. max_texture_side ;
557555
558- if let Some ( font_definitions) = self . memory . new_font_definitions . take ( ) {
559- // New font definition loaded, so we need to reload all fonts.
560- self . fonts = None ;
561- self . font_definitions = font_definitions;
562- #[ cfg( feature = "log" ) ]
563- log:: trace!( "Loading new font definitions" ) ;
564- }
556+ let mut new_font_definitions =
557+ if let Some ( font_definitions) = self . memory . new_font_definitions . take ( ) {
558+ Cow :: Owned ( font_definitions)
559+ } else if let Some ( fonts) = & self . fonts {
560+ Cow :: Borrowed ( fonts. definitions ( ) )
561+ } else {
562+ Cow :: Owned ( FontDefinitions :: default ( ) )
563+ } ;
565564
566565 if !self . memory . add_fonts . is_empty ( ) {
567566 let fonts = self . memory . add_fonts . drain ( ..) ;
567+ let font_definitions = new_font_definitions. to_mut ( ) ;
568568 for font in fonts {
569- self . fonts = None ; // recreate all the fonts
570569 for family in font. families {
571- let fam = self
572- . font_definitions
573- . families
574- . entry ( family. family )
575- . or_default ( ) ;
570+ let fam = font_definitions. families . entry ( family. family ) . or_default ( ) ;
576571 match family. priority {
577572 FontPriority :: Highest => fam. insert ( 0 , font. name . clone ( ) ) ,
578573 FontPriority :: Lowest => fam. push ( font. name . clone ( ) ) ,
579574 }
580575 }
581- self . font_definitions
576+ font_definitions
582577 . font_data
583578 . insert ( font. name , Arc :: new ( font. data ) ) ;
584579 }
@@ -587,28 +582,43 @@ impl ContextImpl {
587582 log:: trace!( "Adding new fonts" ) ;
588583 }
589584
590- let mut is_new = false ;
585+ let mut did_change_fonts = false ;
591586
592- let fonts = self . fonts . get_or_insert_with ( || {
593- #[ cfg( feature = "log" ) ]
594- log:: trace!( "Creating new Fonts for pixels_per_point={pixels_per_point}" ) ;
587+ // If we changed the font definitions this frame, or self.fonts previously did not exist, new_font_definitions
588+ // must be Cow::Owned. Set the font definitions.
589+ let fonts = if let Cow :: Owned ( new_font_definitions) = new_font_definitions {
590+ did_change_fonts = true ;
591+ if let Some ( fonts) = self . fonts . as_mut ( ) {
592+ fonts. set_definitions ( new_font_definitions) ;
593+ did_change_fonts = true ;
594+ fonts
595+ } else {
596+ self . fonts . get_or_insert_with ( || {
597+ #[ cfg( feature = "log" ) ]
598+ log:: trace!( "Creating new FontStore" ) ;
595599
596- is_new = true ;
597- profiling:: scope!( "FontStore::new" ) ;
598- FontStore :: new ( max_texture_side, self . font_definitions . clone ( ) )
599- } ) ;
600+ did_change_fonts = true ;
601+ profiling:: scope!( "FontStore::new" ) ;
602+ FontStore :: new ( max_texture_side, new_font_definitions)
603+ } )
604+ }
605+ } else {
606+ self . fonts
607+ . as_mut ( )
608+ . expect ( "new_font_definitions is borrowed, but we had nowhere to borrow it from" )
609+ } ;
600610
601611 {
602612 profiling:: scope!( "FontStore::begin_pass" ) ;
603613 fonts. begin_pass ( max_texture_side) ;
604614 }
605615
606- if is_new && self . memory . options . preload_font_glyphs {
616+ if did_change_fonts && self . memory . options . preload_font_glyphs {
607617 profiling:: scope!( "preload_font_glyphs" ) ;
608618 // Preload the most common characters for the most common fonts.
609619 // This is not very important to do, but may save a few GPU operations.
610620 for font_id in self . memory . options . style ( ) . text_styles . values ( ) {
611- fonts. font ( font_id ) . preload_common_characters ( ) ;
621+ fonts. preload_common_characters ( font_id ) ;
612622 }
613623 }
614624 }
@@ -1490,11 +1500,10 @@ impl Context {
14901500
14911501 let font_id = TextStyle :: Body . resolve ( & self . style ( ) ) ;
14921502 self . fonts ( |f| {
1493- let font = f. font ( & font_id) ;
1494- font. has_glyphs ( alt)
1495- && font. has_glyphs ( ctrl)
1496- && font. has_glyphs ( shift)
1497- && font. has_glyphs ( mac_cmd)
1503+ f. has_glyphs_for ( & font_id, alt)
1504+ && f. has_glyphs_for ( & font_id, ctrl)
1505+ && f. has_glyphs_for ( & font_id, shift)
1506+ && f. has_glyphs_for ( & font_id, mac_cmd)
14981507 } )
14991508 } ;
15001509
@@ -2904,7 +2913,7 @@ impl Context {
29042913 }
29052914
29062915 fn fonts_tweak_ui ( & self , ui : & mut Ui ) {
2907- let mut font_definitions = self . write ( |ctx| ctx . font_definitions . clone ( ) ) ;
2916+ let mut font_definitions = self . fonts ( |fonts| fonts . definitions ( ) . clone ( ) ) ;
29082917 let mut changed = false ;
29092918
29102919 for ( name, data) in & mut font_definitions. font_data {
0 commit comments