Closed Bug 345751 Opened 18 years ago Closed 18 years ago

Use the new sorted range feature in the selection

Categories

(Core :: Spelling checker, defect)

Product:
Core
Component:
Spelling checker
Version:
1.8 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.8.1beta2

People

(Reporter: brettw, Assigned: brettw)

References

Details

(Keywords: fixed1.8.1, perf)

Attachments

(1 file)

Patch
2.74 KB, patch
roc
: review+
roc
: superreview+
mtschrep
: approval1.8.1+
Details | Diff | Splinter Review 
Bug 345099 introduces faster range intersections in the selection code. Doing this is one of the slowest part of the spellchecker. We should use this capability.
Attached patch Patch — — Splinter Review 

        Attachment #230464 -
        Flags: review?(roc)

    
  
Keywords: perf
Whiteboard: [has patch][needs review]

    
    

    
  
Fixed on trunk.
Whiteboard: [has patch][needs review] → [baking on trunk]

    
  

        Attachment #230464 -
        Flags: approval1.8.1?

    
  
Whiteboard: [baking on trunk] → [needs approval]

    
    

    
  
Comment on attachment 230464 [details] [diff] [review]
Patch

approved by schrep for drivers

        Attachment #230464 -
        Flags: approval1.8.1? → approval1.8.1+

    
    

    
  
Fixed on branch.
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: [needs approval]

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: