Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce memory usages on Praat lexer #1855

Merged
merged 1 commit into from
Dec 3, 2022

Conversation

tancnle
Copy link
Collaborator

@tancnle tancnle commented Aug 8, 2022

This helps to reduce memory consumption by allowing objects to be garbage collected. This can be verified by running rake check:memory task.

retained objects by file
        <snipped>
         15  rouge/lib/rouge/lexers/mosel.rb                                          
         14  ruby/lib/lib/ruby/2.7.0/set.rb                                           
         13  rouge/lib/rouge/lexers/common_lisp.rb                                    
-        13  rouge/lib/rouge/lexers/praat.rb                                          
         13  rouge/lib/rouge/themes/igor_pro.rb                                                                                                                             
         12  rouge/lib/rouge/lexers/haxe.rb                                           
         10  rouge/lib/rouge/guessers/util.rb    
          <snipped>

@tancnle tancnle force-pushed the maint.class-methods-praat-lexer branch 2 times, most recently from 48d39a7 to da47bf8 Compare August 28, 2022 11:27
@tancnle tancnle changed the title Refactor variables to class methods on Praat lexer Reduce memory usages on Praat lexer Aug 28, 2022
@tancnle tancnle force-pushed the maint.class-methods-praat-lexer branch from da47bf8 to cd1eb6a Compare August 29, 2022 08:21
This helps to reduce memory consumption by allowing objects to be
garbage collected. This can be verified by running `rake check:memory`
task.
@tancnle tancnle force-pushed the maint.class-methods-praat-lexer branch from cd1eb6a to f83df91 Compare August 29, 2022 08:58
@tancnle tancnle merged commit 2e0adc3 into rouge-ruby:master Dec 3, 2022
@tancnle tancnle deleted the maint.class-methods-praat-lexer branch December 3, 2022 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants