Osu: Move any transformations added at point of BDL

Created on 19 Dec 2019  路  2Comments  路  Source: ppy/osu

Adding transformations at BDL may never be applied, which obviously causes issues as reported by users, see https://github.com/ppy/osu/issues/6269 & https://github.com/ppy/osu/issues/7288

Should be moved to LoadComplete which is the best point to start adding transformations on as the drawable is fully loaded.

code-quality

Most helpful comment

Potentially another thing to throw a custom analyzer at if that ever happens.

All 2 comments

Potentially another thing to throw a custom analyzer at if that ever happens.

This is a general practice for our use-cases, but is fully supported by the framework - drawables _do_ get a clock inside BDL unlike the mentioned comment states. It is generally seen as unsafe since time may be progressing during the load (e.g. during an async context).

As such, fixes only need to be applied on a per-case basis and we don't need a general issue thread for this.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Axeanz picture Axeanz  路  3Comments

Fatal3ty picture Fatal3ty  路  3Comments

Gabixel picture Gabixel  路  3Comments

Gonzalo-Bruna picture Gonzalo-Bruna  路  3Comments

GameBoyYeet picture GameBoyYeet  路  3Comments