Stop throwing error on space after defined keyword

Author: dtarassenko

src/lint/rules/spacing.rs

@@ -677,8 +677,20 @@ pub struct NspUnaryRule {
 pub type NspUnaryArgs = ZeroRange;
 
 impl NspUnaryArgs {
+    fn is_exception(node: &UnaryExpressionContent) -> bool {
+        // Defined keyword counts as UnaryOp for DLS, but we allow space after it
+        match node.operation.get_token() {
+            Some(token) => match token.kind {
+                TokenKind::Defined => true,
+                _ => false,
+            },
+            None => false,
+        }
+    }
+
     pub fn from_unary_expr(node: &UnaryExpressionContent)
                            -> Option<NspUnaryArgs> {
+        if Self::is_exception(node) { return None; }
         let mut gap = node.range();
         gap.col_start = node.operation.range().col_end;
         gap.col_end = node.expr.range().col_start;

src/lint/rules/tests/spacing/nsp_unary.rs

@@ -43,4 +43,16 @@ method this_is_some_method(conf_object_t *dummy_obj) {
 fn no_space_unary_correct() {
     let rules = set_up();
     assert_snippet(NO_SPACE_UNARY_CORRECT, vec![], &rules);
+}
+
+static UNARY_EXCEPTIONS: &str = "
+method this_is_some_method(conf_object_t *dummy_obj) {
+    if (!defined attr_workaround)
+        return;
+}
+";
+#[test]
+fn unary_exceptions() {
+    let rules = set_up();
+    assert_snippet(UNARY_EXCEPTIONS, vec![], &rules);
 }