Closure fails to format correctly when there are too many generics

[maltekliemann]

I'm using vanilla settings.

let _ = |first, second: SomeType<With, Many, Many, Many, Many, Many, Many, Many, Many, Generic, Arguments>, third| {};

...formats to...

let _ = |first,
         second: SomeType<
    With,
    Many,
    Many,
    Many,
    Many,
    Many,
    Many,
    Many,
    Many,
    Generic,
    Arguments,
>,
         third| {};

Is this expected behavior? I would have expected...

let _ = |first,
         second: SomeType<
             With,
             Many,
             Many,
             Many,
             Many,
             Many,
             Many,
             Many,
             Many,
             Generic,
             Arguments,
         >,
         third| {};

...or (I'm not a big fan of the 1 space indentation for closure arguments)...

let _ = |
    first,
    second: SomeType<
        With,
        Many,
        Many,
        Many,
        Many,
        Many,
        Many,
        Many,
        Many,
        Generic,
        Arguments,
    >,
    third
| {};

[Hugo-Trentesaux]

Might not be a problem of the number of generics, but the length:

// this does not format
let offenders = current_validators
    .into_iter()
    .enumerate()
    .filter_map(|(_, id)| 
    <<Runtime as pallet_im_online::Config>::ValidatorSet as ValidatorSetWithIdentification<
        sp_runtime::AccountId32>>::IdentificationOf::convert(id.clone()).map(|full_id| (id, full_id)))
    .collect::<Vec<IdentificationTuple<Runtime>>>();

// this type alias prevents the formatting bug
type XXX =
    <<Runtime as pallet_im_online::Config>::ValidatorSet as ValidatorSetWithIdentification<
        sp_runtime::AccountId32,
    >>::IdentificationOf;

// this does format
let offenders = current_validators
    .into_iter()
    .enumerate()
    .filter_map(|(_, id)| XXX::convert(id.clone()).map(|full_id| (id, full_id)))
    .collect::<Vec<IdentificationTuple<Runtime>>>();