Fix escaping of term codes in Terminal256 formatter (#1404)

This commit fixes a bug in escaping term codes around `\n` in the Terminal256 formatter.
rouge-ruby · Jan 23, 2020 · 439d9e5 · 439d9e5
1 parent 22ac678
commit 439d9e5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/lib/rouge/formatters/terminal256.rb b/lib/rouge/formatters/terminal256.rb
@@ -85,8 +85,8 @@ def bg
 
         def stream_value(val, &b)
           yield style_string
-          yield val.gsub("\n", "#{reset_string}\n#{style_string}")
-                   .gsub("\e", "\\e")
+          yield val.gsub("\e", "\\e")
+                   .gsub("\n", "#{reset_string}\n#{style_string}")
           yield reset_string
         end
 

diff --git a/spec/lexers/escape_spec.rb b/spec/lexers/escape_spec.rb
@@ -17,10 +17,13 @@
 
   describe 'terminal256' do
     let(:formatter) { Rouge::Formatters::Terminal256.new }
-    let(:text) { %({ "foo": <!\e123!> }) }
+    let(:text) { %({ "foo": \n <!\e123!> }) }
 
     it 'unescapes' do
       assert { result =~ /\e123/ }
+
+      # shouldn't escape the term codes around \n
+      assert { result =~ /\n\e/ }
     end
   end
 end