Make error and warning annotations mandatory in UI tests

This change makes error and warning annotations mandatory in UI tests.
The only exception are tests that use error patterns to match compiler
output and don't have any annotations.

Author: tmiasko

src/test/ui/associated-type-bounds/dyn-lcsit.rs

@@ -2,7 +2,7 @@
 
 #![feature(associated_type_bounds)]
 #![feature(impl_trait_in_bindings)]
-
+//~^ WARNING `impl_trait_in_bindings` is incomplete
 #![allow(non_upper_case_globals)]
 
 use std::ops::Add;

src/test/ui/associated-type-bounds/lcsit.rs

@@ -2,7 +2,7 @@
 
 #![feature(associated_type_bounds)]
 #![feature(impl_trait_in_bindings)]
-
+//~^ WARNING `impl_trait_in_bindings` is incomplete
 #![allow(non_upper_case_globals)]
 
 use std::ops::Add;

src/test/ui/associated-type-bounds/type-alias.rs

@@ -1,19 +1,19 @@
-// build-pass (FIXME(62277): could be check-pass?)
+// check-pass
 
 #![feature(associated_type_bounds)]
 
-type _TaWhere1<T> where T: Iterator<Item: Copy> = T;
-type _TaWhere2<T> where T: Iterator<Item: 'static> = T;
-type _TaWhere3<T> where T: Iterator<Item: 'static> = T;
-type _TaWhere4<T> where T: Iterator<Item: 'static + Copy + Send> = T;
-type _TaWhere5<T> where T: Iterator<Item: for<'a> Into<&'a u8>> = T;
-type _TaWhere6<T> where T: Iterator<Item: Iterator<Item: Copy>> = T;
+type _TaWhere1<T> where T: Iterator<Item: Copy> = T; //~ WARNING type_alias_bounds
+type _TaWhere2<T> where T: Iterator<Item: 'static> = T; //~ WARNING type_alias_bounds
+type _TaWhere3<T> where T: Iterator<Item: 'static> = T; //~ WARNING type_alias_bounds
+type _TaWhere4<T> where T: Iterator<Item: 'static + Copy + Send> = T; //~ WARNING type_alias_bounds
+type _TaWhere5<T> where T: Iterator<Item: for<'a> Into<&'a u8>> = T; //~ WARNING type_alias_bounds
+type _TaWhere6<T> where T: Iterator<Item: Iterator<Item: Copy>> = T; //~ WARNING type_alias_bounds
 
-type _TaInline1<T: Iterator<Item: Copy>> = T;
-type _TaInline2<T: Iterator<Item: 'static>> = T;
-type _TaInline3<T: Iterator<Item: 'static>> = T;
-type _TaInline4<T: Iterator<Item: 'static + Copy + Send>> = T;
-type _TaInline5<T: Iterator<Item: for<'a> Into<&'a u8>>> = T;
-type _TaInline6<T: Iterator<Item: Iterator<Item: Copy>>> = T;
+type _TaInline1<T: Iterator<Item: Copy>> = T; //~ WARNING type_alias_bounds
+type _TaInline2<T: Iterator<Item: 'static>> = T; //~ WARNING type_alias_bounds
+type _TaInline3<T: Iterator<Item: 'static>> = T; //~ WARNING type_alias_bounds
+type _TaInline4<T: Iterator<Item: 'static + Copy + Send>> = T; //~ WARNING type_alias_bounds
+type _TaInline5<T: Iterator<Item: for<'a> Into<&'a u8>>> = T; //~ WARNING type_alias_bounds
+type _TaInline6<T: Iterator<Item: Iterator<Item: Copy>>> = T; //~ WARNING type_alias_bounds
 
 fn main() {}

src/test/ui/async-await/issues/issue-54752-async-block.rs

@@ -4,3 +4,4 @@
 // pp-exact
 
 fn main() { let _a = (async  { }); }
+//~^ WARNING unnecessary parentheses around assigned value

src/test/ui/block-expr-precedence.rs

@@ -58,4 +58,5 @@ pub fn main() {
   if (true) { 12; } {-num};
   if (true) { 12; }; {-num};
   if (true) { 12; };;; -num;
+  //~^ WARNING unnecessary trailing semicolons
 }

src/test/ui/codemap_tests/unicode_3.rs

@@ -1,6 +1,6 @@
 // build-pass (FIXME(62277): could be check-pass?)
 
 fn main() {
-    let s = "ZͨA͑ͦ͒͋ͤ͑̚L̄͑͋Ĝͨͥ̿͒̽̈́Oͥ͛ͭ!̏"; while true { break; }
+    let s = "ZͨA͑ͦ͒͋ͤ͑̚L̄͑͋Ĝͨͥ̿͒̽̈́Oͥ͛ͭ!̏"; while true { break; } //~ WARNING while_true
     println!("{}", s);
 }

src/test/ui/consts/const-eval/const_fn_ptr.rs

@@ -6,18 +6,18 @@ fn double(x: usize) -> usize { x * 2 }
 const fn double_const(x: usize) -> usize { x * 2 }
 
 const X: fn(usize) -> usize = double;
-const X_const: fn(usize) -> usize = double_const;
+const X_CONST: fn(usize) -> usize = double_const;
 
 const fn bar(x: usize) -> usize {
-    X(x)
+    X(x) //~ WARNING skipping const checks
 }
 
 const fn bar_const(x: usize) -> usize {
-    X_const(x)
+    X_CONST(x) //~ WARNING skipping const checks
 }
 
 const fn foo(x: fn(usize) -> usize, y: usize)  -> usize {
-    x(y)
+    x(y) //~ WARNING skipping const checks
 }
 
 fn main() {

src/test/ui/consts/const-eval/const_fn_ptr.stderr

@@ -7,7 +7,7 @@ LL |     X(x)
 warning: skipping const checks
   --> $DIR/const_fn_ptr.rs:16:5
    |
-LL |     X_const(x)
+LL |     X_CONST(x)
    |     ^^^^^^^^^^
 
 warning: skipping const checks
@@ -16,11 +16,3 @@ warning: skipping const checks
 LL |     x(y)
    |     ^^^^
 
-warning: constant `X_const` should have an upper case name
-  --> $DIR/const_fn_ptr.rs:9:7
-   |
-LL | const X_const: fn(usize) -> usize = double_const;
-   |       ^^^^^^^ help: convert the identifier to upper case: `X_CONST`
-   |
-   = note: `#[warn(non_upper_case_globals)]` on by default
-

src/test/ui/consts/const-eval/issue-64970.rs

@@ -10,6 +10,6 @@ fn foo(mut n: i32) {
     }
 
     if n > 0i32 {
-        1i32 / n;
+        let _ = 1i32 / n;
     }
 }