Skip to content

Comments

Release 0.6.0#30

Open
harvimt wants to merge 15 commits intoDan-Q:masterfrom
harvimt:release-0.6.0
Open

Release 0.6.0#30
harvimt wants to merge 15 commits intoDan-Q:masterfrom
harvimt:release-0.6.0

Conversation

@harvimt
Copy link

@harvimt harvimt commented Sep 16, 2017

I'm not sure why the decompiler was disabled on windows, it works just
fine.

The only reason I can think of is that it requires nokogiri which is a C
extension, but the dep on nokogiri isn't exempted, so windows users who
did `gem install twee2` were getting it anyway.

Additionally, there are precompiled versions of nokogiri available for
windows now, so you don't need a C compiler.

Furthermore, RubyInstaller can install the C compilers now.
Ruby's gsub method is a little weird in how it handles substitution
strings and there's no way to turn it off.

I could escape the substitution string, but the translation is something
like backslash followed by a number, & or single quote

something like:

def escape_sub(s) s.gsub(/\\[0-9&']/, '\\\&'); end

this way strikes me as easier to reason about (though not necessarily
any less convoluted)
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