ZequanWu wrote:

> @ZequanWu would either of these work for you?
> 
> ```c++
>  DCHECK_OK(foo); // 
> expected-error@components/reporting/util/status_macros.h:* 
> {{{CHECK,DCHECK,ASSERT,EXPECT}_OK do not accept a type other than Status or 
> StatusOr.}}}
> ```
> 
> or
> 
> ```c++
>  DCHECK_OK(foo); // 
> expected-error-re@components/reporting/util/status_macros.h:* 
> {{{CHECK,DCHECK,ASSERT,EXPECT}_OK do not accept a type other than Status or 
> StatusOr.}}
> ```

Thanks, it's a simple fix, just adding a space between 2nd and 3rd braces:
```
DCHECK_OK(foo);  // expected-error@components/reporting/util/status_macros.h:* 
{{ {CHECK,DCHECK,ASSERT,EXPECT}_OK do not accept a type other than Status or 
StatusOr.}}
```

https://github.com/llvm/llvm-project/pull/77326
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to