Skip to content

Commit 4b6017f

Browse files
sapersaper
committed
Remove extra semicolons
In clang 3.6.x, -Wextra-semi warnings are issued with -Wall: 21:37 <+drew> ./ast.hpp:1785:31: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1804:31: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1823:31: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1824:34: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1825:33: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1853:31: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1854:38: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1895:31: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1896:38: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1988:41: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1989:43: warning: extra ';' inside a class [-Wextra-semi] 21:37 <+drew> ./ast.hpp:1990:42: warning: extra ';' inside a class [-Wextra-semi]
1 parent 7db8a5b commit 4b6017f

File tree

9 files changed

+249
-249
lines changed

9 files changed

+249
-249
lines changed

‎ast.hpp

Lines changed: 213 additions & 213 deletions
Large diffs are not rendered by default.

‎context.hpp

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -85,29 +85,29 @@ namespace Sass {
8585
size_t precision; // precision for outputting fractional numbers
8686

8787
KWD_ARG_SET(Data) {
88-
KWD_ARG(Data, struct Sass_Options*, c_options);
89-
KWD_ARG(Data, struct Sass_Compiler*, c_compiler);
90-
KWD_ARG(Data, const char*, source_c_str);
91-
KWD_ARG(Data, string, entry_point);
92-
KWD_ARG(Data, string, input_path);
93-
KWD_ARG(Data, string, output_path);
94-
KWD_ARG(Data, string, indent);
95-
KWD_ARG(Data, string, linefeed);
96-
KWD_ARG(Data, const char*, include_paths_c_str);
97-
KWD_ARG(Data, const char*, plugin_paths_c_str);
98-
// KWD_ARG(Data, const char**, include_paths_array);
99-
// KWD_ARG(Data, const char**, plugin_paths_array);
100-
KWD_ARG(Data, vector<string>, include_paths);
101-
KWD_ARG(Data, vector<string>, plugin_paths);
102-
KWD_ARG(Data, bool, source_comments);
103-
KWD_ARG(Data, Output_Style, output_style);
104-
KWD_ARG(Data, string, source_map_file);
105-
KWD_ARG(Data, string, source_map_root);
106-
KWD_ARG(Data, bool, omit_source_map_url);
107-
KWD_ARG(Data, bool, is_indented_syntax_src);
108-
KWD_ARG(Data, size_t, precision);
109-
KWD_ARG(Data, bool, source_map_embed);
110-
KWD_ARG(Data, bool, source_map_contents);
88+
KWD_ARG(Data, struct Sass_Options*, c_options)
89+
KWD_ARG(Data, struct Sass_Compiler*, c_compiler)
90+
KWD_ARG(Data, const char*, source_c_str)
91+
KWD_ARG(Data, string, entry_point)
92+
KWD_ARG(Data, string, input_path)
93+
KWD_ARG(Data, string, output_path)
94+
KWD_ARG(Data, string, indent)
95+
KWD_ARG(Data, string, linefeed)
96+
KWD_ARG(Data, const char*, include_paths_c_str)
97+
KWD_ARG(Data, const char*, plugin_paths_c_str)
98+
// KWD_ARG(Data, const char**, include_paths_array)
99+
// KWD_ARG(Data, const char**, plugin_paths_array)
100+
KWD_ARG(Data, vector<string>, include_paths)
101+
KWD_ARG(Data, vector<string>, plugin_paths)
102+
KWD_ARG(Data, bool, source_comments)
103+
KWD_ARG(Data, Output_Style, output_style)
104+
KWD_ARG(Data, string, source_map_file)
105+
KWD_ARG(Data, string, source_map_root)
106+
KWD_ARG(Data, bool, omit_source_map_url)
107+
KWD_ARG(Data, bool, is_indented_syntax_src)
108+
KWD_ARG(Data, size_t, precision)
109+
KWD_ARG(Data, bool, source_map_embed)
110+
KWD_ARG(Data, bool, source_map_contents)
111111
};
112112

113113
Context(Data);

‎emitter.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ namespace Sass {
1313

1414
public:
1515
Emitter(Context* ctx);
16-
virtual ~Emitter() { };
16+
virtual ~Emitter() { }
1717

1818
protected:
1919
OutputBuffer wbuf;

‎environment.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ namespace Sass {
1919
class Environment {
2020
// TODO: test with unordered_map
2121
map<string, T> local_frame_;
22-
ADD_PROPERTY(Environment*, parent);
22+
ADD_PROPERTY(Environment*, parent)
2323

2424
public:
2525
Memory_Manager<AST_Node> mem;

‎extend.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ namespace Sass {
2222
Context& ctx;
2323
ExtensionSubsetMap& subset_map;
2424

25-
void fallback_impl(AST_Node* n) { };
25+
void fallback_impl(AST_Node* n) { }
2626

2727
public:
2828
Extend(Context&, ExtensionSubsetMap&);

‎plugins.hpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,9 @@ namespace Sass {
4242
size_t load_plugins(const string& path);
4343

4444
public: // public accessors
45-
const vector<Sass_Importer_Entry> get_headers(void) { return headers; };
46-
const vector<Sass_Importer_Entry> get_importers(void) { return importers; };
47-
const vector<Sass_Function_Entry> get_functions(void) { return functions; };
45+
const vector<Sass_Importer_Entry> get_headers(void) { return headers; }
46+
const vector<Sass_Importer_Entry> get_importers(void) { return importers; }
47+
const vector<Sass_Function_Entry> get_functions(void) { return functions; }
4848

4949
private: // private vars
5050
vector<Sass_Importer_Entry> headers;
@@ -55,4 +55,4 @@ namespace Sass {
5555

5656
}
5757

58-
#endif
58+
#endif

‎position.hpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ namespace Sass {
3636
// friend ostream& operator<<(ostream& strm, const Offset& off);
3737

3838
public:
39-
Offset off() { return *this; };
39+
Offset off() { return *this; }
4040

4141
public:
4242
size_t line;
@@ -109,9 +109,9 @@ namespace Sass {
109109
ParserState(string path, const char* src, Token token, Position position, Offset offset = Offset(0, 0));
110110

111111
public: // down casts
112-
Offset off() { return *this; };
113-
Position pos() { return *this; };
114-
ParserState pstate() { return *this; };
112+
Offset off() { return *this; }
113+
Position pos() { return *this; }
114+
ParserState pstate() { return *this; }
115115

116116
public:
117117
string path;

‎remove_placeholders.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ namespace Sass {
1818

1919
Context& ctx;
2020

21-
void fallback_impl(AST_Node* n) {};
21+
void fallback_impl(AST_Node* n) {}
2222

2323
public:
2424
Remove_Placeholders(Context&);

‎units.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ namespace Sass {
7777
ss << "'" << unit_to_string(a) << "' and ";
7878
ss << "'" << unit_to_string(b) << "'";
7979
msg = ss.str().c_str();
80-
};
80+
}
8181
virtual const char* what() const throw()
8282
{
8383
return msg;

0 commit comments

Comments
 (0)