fmt

Author: y21

.github/workflows/clippy_changelog.yml

@@ -24,7 +24,7 @@ jobs:
     - name: Check Changelog
       if: ${{ github.event_name == 'pull_request' }}
       run: |
-        body=$(curl -H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" -s "https://api.github.com/repos/rust-lang/rust-clippy/pulls/$PR_NUMBER" | \
+        body=$(curl -H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" -s "https://api.github.com/repos/${{ github.repository_owner }}/rust-clippy/pulls/$PR_NUMBER" | \
           python -c "import sys, json; print(json.load(sys.stdin)['body'])")
         output=$(awk '/^changelog:\s*\S/ && !/changelog: \[.*\]: your change/' <<< "$body" | sed "s/changelog:\s*//g")
         if [ -z "$output" ]; then

clippy_lints/src/vec.rs

@@ -61,12 +61,13 @@ enum VecState {
 pub struct UselessVec {
     too_large_for_stack: u64,
     msrv: Msrv,
-    /// Maps from a `vec![]` source callsite invocation span to the "state" (i.e., whether we can emit a warning there or not).
+    /// Maps from a `vec![]` source callsite invocation span to the "state" (i.e., whether we can
+    /// emit a warning there or not).
     ///
-    /// The purpose of this is to buffer lints up until `check_expr_post` so that we can cancel a lint while visiting,
-    /// because a `vec![]` invocation span can appear multiple times when it is passed as a macro argument,
-    /// once in a context that doesn't require a `Vec<_>` and another time that does. Consider:
-    /// ```
+    /// The purpose of this is to buffer lints up until `check_expr_post` so that we can cancel a
+    /// lint while visiting, because a `vec![]` invocation span can appear multiple times when
+    /// it is passed as a macro argument, once in a context that doesn't require a `Vec<_>` and
+    /// another time that does. Consider: ```
     /// macro_rules! m {
     ///     ($v:expr) => {
     ///         let a = $v;
@@ -75,8 +76,9 @@ pub struct UselessVec {
     /// }
     /// m!(vec![1, 2]);
     /// ```
-    /// The macro invocation expands to two `vec![1, 2]` invocations. If we eagerly suggest changing it to an array,
-    /// we get a false positive warning on the `$v.push(3)` which really requires `$v` to be a vector.
+    /// The macro invocation expands to two `vec![1, 2]` invocations. If we eagerly suggest changing
+    /// it to an array, we get a false positive warning on the `$v.push(3)` which really
+    /// requires `$v` to be a vector.
     span_to_state: BTreeMap<Span, VecState>,
     allow_in_test: bool,
 }
@@ -93,7 +95,8 @@ impl UselessVec {
 }
 
 enum VecToArray {
-    /// Expression does not need to be a `Vec<_>` and its type can be changed to an array (or slice).
+    /// Expression does not need to be a `Vec<_>` and its type can be changed to an array (or
+    /// slice).
     Possible,
     /// Expression must be a `Vec<_>`. Type cannot change.
     Impossible,