record-dot-preprocessor updates are lost sometimes

[neongreen]

Input:

{-# OPTIONS_GHC -fplugin=RecordDotPreprocessor #-}

foo = a{b = c
 }

fooplus f n = f{foo = f.bar + n}

Output:

{-# OPTIONS_GHC -fplugin=RecordDotPreprocessor #-}

foo =
  a { b = c
    }

fooplus f n = f{foo = f.bar + n}

The right result should be:

{-# OPTIONS_GHC -fplugin=RecordDotPreprocessor #-}

foo =
  a{b = c
   }

fooplus f n = f{foo = f.bar + n}

[neongreen]

The bug is somewhere in https://github.com/tweag/ormolu/pull/499.

[neongreen]

Hm, it's fishier than I thought. Input:

{-# OPTIONS_GHC -fplugin=RecordDotPreprocessor #-}

foo =   req
          { a = b 
          }

Output:

{-# OPTIONS_GHC -fplugin=RecordDotPreprocessor #-}

foo =
  req{ a = b
     }

But now if you remove one of the spaces before req, you don't get the RDP update anymore. The correct behavior is that there should not an RDP update in either case. Input:

{-# OPTIONS_GHC -fplugin=RecordDotPreprocessor #-}

foo =  req
          { a = b 
          }

Output:

{-# OPTIONS_GHC -fplugin=RecordDotPreprocessor #-}

foo =
  req
    { a = b
    }

[neongreen]

@Avi-D-coder can you take another look at this?

[neongreen]

Oops, mis-copied the last example. I edited the comment.

[cattingcat]

@neongreen Workin on fix here: https://github.com/tweag/ormolu/pull/663