Skip to content

Conversation

@Tyriar
Copy link
Member

@Tyriar Tyriar commented Jan 31, 2026

Commenting out #5645 optimization:

   Context "out-test/benchmark/Event.benchmark.js"
      Context "Emitter.fire()"
         Context "0 listeners"
            Case "#1" : 5 runs - average runtime: 5.62 ms
         Context "1 listener"
            Case "#1" : 5 runs - average runtime: 17.27 ms
         Context "2 listeners"
            Case "#1" : 5 runs - average runtime: 26.54 ms
         Context "5 listeners"
            Case "#1" : 5 runs - average runtime: 57.61 ms

main:

   Context "out-test/benchmark/Event.benchmark.js"
      Context "Emitter.fire()"
         Context "0 listeners"
            Case "#1" : 5 runs - average runtime: 2.06 ms
         Context "1 listener"
            Case "#1" : 5 runs - average runtime: 9.72 ms
         Context "2 listeners"
            Case "#1" : 5 runs - average runtime: 26.26 ms
         Context "5 listeners"
            Case "#1" : 5 runs - average runtime: 56.72 ms

@jerch fixed 😃

@Tyriar Tyriar added this to the 6.1.0 milestone Jan 31, 2026
@Tyriar Tyriar self-assigned this Jan 31, 2026
@Tyriar Tyriar enabled auto-merge January 31, 2026 23:31
@Tyriar Tyriar merged commit 77036d3 into xtermjs:master Jan 31, 2026
12 checks passed
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.

1 participant