4 Commits

• Remove unused virtual keywords

  Added these in a previous iteration, where I had the mania variant inherit this class.
  
  No longer necessary as `IHasColumn` was used to make this check more generic.

  Naxess · 2021-04-27 02:23:06 +02:00
  9e49ecb573
• Fix accessibility of areConcurrent
  Naxess · 2021-04-27 01:24:38 +02:00
  a3c1b1fd52
• Rename CheckUnsnaps -> CheckUnsnappedObjects

  Will potentially have `CheckUnsnappedKiai` or similar later, so this is worth specifying.
  
  Also consistent with `CheckConcurrentObjects`, which will likely have a `CheckConcurrentLines` later.

  Naxess · 2021-04-26 20:32:44 +02:00
  ce258febf6
• Add concurrent objects check

  Here we use `IHasColumn` to support rulesets with columns, and so I moved that interface out into `osu.Game` from `osu.Game.Rulesets.Mania`.
  
  We also use the same threshold as the unsnap check to ensure that no problems slip through. Specifically where an object is simultaneously not concurrent and not unsnapped but still on the same tick.

  Naxess · 2021-04-26 20:17:18 +02:00
  a3570e18dd