BatchManager: Unset textureRegion? about shapedrawer HOT 4 CLOSED

Also, it is not possible to instantiate a ShapeDrawer without textureRegion or textureRegion = null.
This is related to the same problem.

from shapedrawer.

OK that's a fair enough use case, you might also want to use different Texture Regions from different textures to optimise flushes.

You can make a PR if you like, otherwise I'll do it this week. It should also throw an exception (IllegalStateException?) when trying to use it without a Texture Region. I guess the setter would then just become

    public TextureRegion setTextureRegion(TextureRegion region) {
        TextureRegion oldRegion = this.r;
        this.r = region;
        if (r != null) {
            float u = 0.5f * (r.getU() + r.getU2());
            float v = 0.5f * (r.getV() + r.getV2());
            for (int i = 0; i < verts.length; i += VERTEX_SIZE) {
                verts[i + SpriteBatch.U1] = u;
                verts[i + SpriteBatch.V1] = v;
            }
        }
        return oldRegion;
    }

and you could add a getter and a constructor which takes only a Batch.

from shapedrawer.

@earlygrey Hey, I'll create a PR soon.

from shapedrawer.

Closed with #34

from shapedrawer.