这是indexloc提供的服务,不要输入任何密码
Skip to content

Should cullMode require "none" for points and lines? #2863

@toji

Description

@toji

This was an inline issue in the spec, and seems trivial enough to resolve that I'm writing it up as a hopeful tacit resolution candiate. Currently the GPUPrimitiveState has a cullMode that defaults to "none", but the other possible cull modes ("front" and "back") don't apply if the topology is a point or line type.

In some other cases, such as stripIndexFormat, we've required that the value be left undefined when it doesn't apply, so we could do the same here.

On the other hand, there's no particular issue with developers specifying a cull mode in these cases. It's not validated on any of the native APIs, doesn't cause adverse effects when rendering, and the default is such that developers who specify it are really just making more work for themselves.

My preference is to simply not validate it. There's no technical reason to block developers from setting this, and doing so mildly breaks backwards compatibility.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions