Skip to content

Commit

Permalink
small cleanup and quash a compiler warning
Browse files Browse the repository at this point in the history
- remove unneeded require path from gemspec
- remove unneeded ruby.h include from better_html.h
- fix the C function parameters for Parser#errors, removing unused
  parameter that is generating warnings
  • Loading branch information
flavorjones committed Aug 12, 2022
1 parent bddb2cc commit 6e9fe5a
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion better_html.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ Gem::Specification.new do |s|
s.extensions = ['ext/better_html_ext/extconf.rb']
s.files = Dir["{app,config,db,lib,ext}/**/*", "MIT-LICENSE", "Rakefile", "README.rdoc"]
s.test_files = Dir["test/**/*"]
s.require_paths = ["lib", "ext"]
s.require_paths = ["lib"]

s.add_dependency 'ast', '~> 2.0'
s.add_dependency 'erubi', '~> 1.4'
Expand Down
1 change: 0 additions & 1 deletion ext/better_html_ext/better_html.h
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
#include <ruby.h>
#include "html_tokenizer.h"
2 changes: 1 addition & 1 deletion ext/better_html_ext/parser.c
Original file line number Diff line number Diff line change
Expand Up @@ -734,7 +734,7 @@ static VALUE create_parser_error(struct parser_document_error_t *error)
return rb_class_new_instance(4, args, klass);
}

static VALUE parser_errors_method(VALUE self, VALUE error_p)
static VALUE parser_errors_method(VALUE self)
{
struct parser_t *parser = NULL;
VALUE list;
Expand Down

0 comments on commit 6e9fe5a

Please sign in to comment.