Merge pull request #21 from Shopify/fix-unused-warnings

Enable -Wextra
Shopify · Jan 31, 2015 · 32358a1 · 32358a1
2 parents 8ca9317 + 05fd88a
commit 32358a1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/ext/liquid_c/extconf.rb b/ext/liquid_c/extconf.rb
@@ -1,5 +1,5 @@
 require 'mkmf'
-$CFLAGS << ' -Wall -Werror'
+$CFLAGS << ' -Wall -Werror -Wextra -Wno-unused-parameter -Wno-missing-field-initializers'
 compiler = RbConfig::MAKEFILE_CONFIG['CC']
 if ENV['DEBUG'] == 'true' && compiler =~ /gcc|g\+\+/
   $CFLAGS << ' -fbounds-check'

diff --git a/liquid-c.gemspec b/liquid-c.gemspec
@@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_dependency 'liquid', '~> 3.0.0'
+  spec.add_dependency 'liquid', '>= 3.0.0'
 
   spec.add_development_dependency "bundler", "~> 1.5"
   spec.add_development_dependency "rake"