Skip to content

Commit 5b25c7d

Browse files
committed
parser-cov: align the columns in KeyEventDigger ctor
... to improve code readability Related: https://issues.redhat.com/browse/OSH-552
1 parent 250e301 commit 5b25c7d

File tree

1 file changed

+31
-31
lines changed

1 file changed

+31
-31
lines changed

src/lib/parser-cov.cc

+31-31
Original file line numberDiff line numberDiff line change
@@ -227,44 +227,44 @@ KeyEventDigger::KeyEventDigger():
227227
d(new Private)
228228
{
229229
// register checker-specific key events
230-
d->hMap["BAD_CHECK_OF_WAIT_COND"].insert("wait_cond_improperly_checked");
231-
d->hMap["BAD_LOCK_OBJECT"] .insert("boxed_lock");
232-
d->hMap["BAD_LOCK_OBJECT"] .insert("lock_on_assigned_field");
233-
d->hMap["BAD_LOCK_OBJECT"] .insert("single_thread_lock");
234-
d->hMap["CALL_SUPER"] .insert("missing_super_call");
235-
d->hMap["CHECKED_RETURN"] .insert("check_return");
230+
d->hMap["BAD_CHECK_OF_WAIT_COND"] .insert("wait_cond_improperly_checked");
231+
d->hMap["BAD_LOCK_OBJECT"] .insert("boxed_lock");
232+
d->hMap["BAD_LOCK_OBJECT"] .insert("lock_on_assigned_field");
233+
d->hMap["BAD_LOCK_OBJECT"] .insert("single_thread_lock");
234+
d->hMap["CALL_SUPER"] .insert("missing_super_call");
235+
d->hMap["CHECKED_RETURN"] .insert("check_return");
236236
d->hMap["CONSTANT_EXPRESSION_RESULT"].insert("extra_high_bits");
237237
d->hMap["CONSTANT_EXPRESSION_RESULT"].insert("logical_vs_bitwise");
238238
d->hMap["CONSTANT_EXPRESSION_RESULT"].insert("missing_parentheses");
239239
d->hMap["CONSTANT_EXPRESSION_RESULT"].insert("operator_confusion");
240240
d->hMap["CONSTANT_EXPRESSION_RESULT"].insert("pointless_expression");
241241
d->hMap["CONSTANT_EXPRESSION_RESULT"].insert("result_independent_of_operands");
242242
d->hMap["CONSTANT_EXPRESSION_RESULT"].insert("same_on_both_sides");
243-
d->hMap["EXPLICIT_THIS_EXPECTED"].insert("implicit_this_used");
244-
d->hMap["FORWARD_NULL"] .insert("deref_parm");
245-
d->hMap["FORWARD_NULL"] .insert("dereference");
246-
d->hMap["FORWARD_NULL"] .insert("var_deref_op");
247-
d->hMap["FORWARD_NULL"] .insert("var_deref_model");
248-
d->hMap["LOCK"] .insert("double_lock");
249-
d->hMap["LOCK"] .insert("double_unlock");
250-
d->hMap["MISSING_BREAK"] .insert("unterminated_case");
251-
d->hMap["NESTING_INDENT_MISMATCH"].insert("dangling_else");
252-
d->hMap["NESTING_INDENT_MISMATCH"].insert("multi_stmt_macro");
253-
d->hMap["NESTING_INDENT_MISMATCH"].insert("on_same_line");
254-
d->hMap["NESTING_INDENT_MISMATCH"].insert("uncle");
255-
d->hMap["ORDER_REVERSAL"] .insert("lock_acquire");
256-
d->hMap["OVERRUN_STATIC"] .insert("index_parm");
257-
d->hMap["OVERRUN_STATIC"] .insert("overrun-buffer-arg");
258-
d->hMap["OVERRUN_STATIC"] .insert("overrun-local");
259-
d->hMap["STREAM_FORMAT_STATE"] .insert("format_changed");
260-
d->hMap["UNINIT"] .insert("uninit_use");
261-
d->hMap["UNINIT"] .insert("uninit_use_in_call");
262-
d->hMap["UNINIT_CTOR"] .insert("uninit_member");
263-
d->hMap["USE_AFTER_FREE"] .insert("deref_after_free");
264-
d->hMap["USE_AFTER_FREE"] .insert("deref_arg");
265-
d->hMap["USE_AFTER_FREE"] .insert("double_free");
266-
d->hMap["USE_AFTER_FREE"] .insert("pass_freed_arg");
267-
d->hMap["USE_AFTER_FREE"] .insert("use_after_free");
243+
d->hMap["EXPLICIT_THIS_EXPECTED"] .insert("implicit_this_used");
244+
d->hMap["FORWARD_NULL"] .insert("deref_parm");
245+
d->hMap["FORWARD_NULL"] .insert("dereference");
246+
d->hMap["FORWARD_NULL"] .insert("var_deref_op");
247+
d->hMap["FORWARD_NULL"] .insert("var_deref_model");
248+
d->hMap["LOCK"] .insert("double_lock");
249+
d->hMap["LOCK"] .insert("double_unlock");
250+
d->hMap["MISSING_BREAK"] .insert("unterminated_case");
251+
d->hMap["NESTING_INDENT_MISMATCH"] .insert("dangling_else");
252+
d->hMap["NESTING_INDENT_MISMATCH"] .insert("multi_stmt_macro");
253+
d->hMap["NESTING_INDENT_MISMATCH"] .insert("on_same_line");
254+
d->hMap["NESTING_INDENT_MISMATCH"] .insert("uncle");
255+
d->hMap["ORDER_REVERSAL"] .insert("lock_acquire");
256+
d->hMap["OVERRUN_STATIC"] .insert("index_parm");
257+
d->hMap["OVERRUN_STATIC"] .insert("overrun-buffer-arg");
258+
d->hMap["OVERRUN_STATIC"] .insert("overrun-local");
259+
d->hMap["STREAM_FORMAT_STATE"] .insert("format_changed");
260+
d->hMap["UNINIT"] .insert("uninit_use");
261+
d->hMap["UNINIT"] .insert("uninit_use_in_call");
262+
d->hMap["UNINIT_CTOR"] .insert("uninit_member");
263+
d->hMap["USE_AFTER_FREE"] .insert("deref_after_free");
264+
d->hMap["USE_AFTER_FREE"] .insert("deref_arg");
265+
d->hMap["USE_AFTER_FREE"] .insert("double_free");
266+
d->hMap["USE_AFTER_FREE"] .insert("pass_freed_arg");
267+
d->hMap["USE_AFTER_FREE"] .insert("use_after_free");
268268

269269
// we use COMPILER_WARNING as checker for compiler errors/warnings
270270
d->hMap["COMPILER_WARNING"] .insert("error");

0 commit comments

Comments
 (0)