Skip to content

FastMath.toMultipleOf should be renamed to FastMath.alignToPowerOfTwo #2392

@riccardobl

Description

@riccardobl

See: #1782 (comment)

The FastMath.toMultipleOf , used by the ubo implementation, only works for p that is a power of two, this can be confusing for devs looking to use FastMath apis in their code.

It should be renamed to alignToPowerOfTwo, and a generic implementation for toMultipleOf should be provided in place of the current one.

Metadata

Metadata

Labels

defectSomething that is supposed to work, but doesn't. Less severe than a "bug"

Type

No type

Projects

Status

tracked

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions