rjkroege / edwood

Go version of Plan9 Acme Editor
Other
387 stars 34 forks source link

no-conversion observer to frame #437

Closed rjkroege closed 2 years ago

rjkroege commented 2 years ago

Change the file.BufferObserver interface to eliminate all unnecessary []byte-[]rune conversions down into frame.

codecov[bot] commented 2 years ago

Codecov Report

Merging #437 (7ebb6c7) into master (73458e1) will increase coverage by 0.02%. The diff coverage is 91.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #437      +/-   ##
==========================================
+ Coverage   56.89%   56.91%   +0.02%     
==========================================
  Files          54       54              
  Lines       10593    10598       +5     
==========================================
+ Hits         6027     6032       +5     
  Misses       4137     4137              
  Partials      429      429              
Impacted Files Coverage Δ
file/offset_tuple.go 66.66% <80.00%> (+66.66%) :arrow_up:
file/buffer.go 86.68% <87.50%> (-0.04%) :arrow_down:
text.go 48.74% <87.50%> (+0.09%) :arrow_up:
file/observable_editable_buffer.go 75.24% <90.90%> (+0.24%) :arrow_up:
file/buffer_adapter.go 89.79% <100.00%> (-2.40%) :arrow_down:
frame/insert.go 85.33% <100.00%> (ø)
tagindex.go 85.13% <100.00%> (+0.30%) :arrow_up:
wind.go 75.16% <100.00%> (+0.16%) :arrow_up:
regexp/regexp.go 9.82% <0.00%> (+0.37%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 73458e1...7ebb6c7. Read the comment docs.