[Bf-blender-cvs] [8849bed6719] master: Revert "PyAPI: GPU Shader: add 'state' parameter to uniform sampler"
Germano Cavalcante
noreply at git.blender.org
Thu Sep 2 15:32:28 CEST 2021
Commit: 8849bed67199f9f0ceb6ea6c92feec2281bbb9af
Author: Germano Cavalcante
Date: Thu Sep 2 10:18:44 2021 -0300
Branches: master
https://developer.blender.org/rB8849bed67199f9f0ceb6ea6c92feec2281bbb9af
Revert "PyAPI: GPU Shader: add 'state' parameter to uniform sampler"
This reverts commit 2aad8fc7bc2a45d5f749430c7cb9b82b9f6c9e49.
It was a commit without proper review.
A better API needs to be discussed.
===================================================================
M source/blender/python/gpu/gpu_py_shader.c
===================================================================
diff --git a/source/blender/python/gpu/gpu_py_shader.c b/source/blender/python/gpu/gpu_py_shader.c
index d2167f2f102..95e505b1343 100644
--- a/source/blender/python/gpu/gpu_py_shader.c
+++ b/source/blender/python/gpu/gpu_py_shader.c
@@ -76,21 +76,6 @@ static const struct PyC_StringEnumItems pygpu_shader_config_items[] = {
{0, NULL},
};
-static const struct PyC_FlagSet pygpu_texture_samplerstate_items[] = {
- {GPU_SAMPLER_DEFAULT, "DEFAULT"},
- {GPU_SAMPLER_FILTER, "FILTER"},
- {GPU_SAMPLER_MIPMAP, "MIPMAP"},
- {GPU_SAMPLER_REPEAT_S, "REPEAT_S"},
- {GPU_SAMPLER_REPEAT_T, "REPEAT_T"},
- {GPU_SAMPLER_REPEAT_R, "REPEAT_R"},
- {GPU_SAMPLER_CLAMP_BORDER, "CLAMP_BORDER"},
- {GPU_SAMPLER_COMPARE, "COMPARE"},
- {GPU_SAMPLER_ANISO, "ANISO"},
- {GPU_SAMPLER_ICON, "ICON"},
- {GPU_SAMPLER_REPEAT, "REPEAT"},
- {0, NULL},
-};
-
static int pygpu_shader_uniform_location_get(GPUShader *shader,
const char *name,
const char *error_prefix)
@@ -507,53 +492,25 @@ static PyObject *pygpu_shader_uniform_int(BPyGPUShader *self, PyObject *args)
}
PyDoc_STRVAR(pygpu_shader_uniform_sampler_doc,
- ".. method:: uniform_sampler(name, texture, state={'DEFAULT'})\n"
+ ".. method:: uniform_sampler(name, texture)\n"
"\n"
- " Specify the texture and state for an uniform sampler in the current GPUShader.\n"
+ " Specify the value of a texture uniform variable for the current GPUShader.\n"
"\n"
" :param name: name of the uniform variable whose texture is to be specified.\n"
" :type name: str\n"
" :param texture: Texture to attach.\n"
- " :type texture: :class:`gpu.types.GPUTexture`\n"
- " :param state: set of values in:\n"
- "\n"
- " - ``DEFAULT``\n"
- " - ``FILTER``\n"
- " - ``MIPMAP``\n"
- " - ``REPEAT_S``\n"
- " - ``REPEAT_T``\n"
- " - ``REPEAT_R``\n"
- " - ``CLAMP_BORDER``\n"
- " - ``COMPARE``\n"
- " - ``ANISO``\n"
- " - ``ICON``\n"
- " - ``REPEAT``\n"
- " :type state: set\n");
-static PyObject *pygpu_shader_uniform_sampler(BPyGPUShader *self, PyObject *args, PyObject *kwds)
+ " :type texture: :class:`gpu.types.GPUTexture`\n");
+static PyObject *pygpu_shader_uniform_sampler(BPyGPUShader *self, PyObject *args)
{
const char *name;
BPyGPUTexture *py_texture;
- PyObject *py_samplerstate = NULL;
-
- static const char *_keywords[] = {"name", "texture", "state", NULL};
- static _PyArg_Parser _parser = {"sO!|$O:uniform_sampler", _keywords, 0};
- if (!_PyArg_ParseTupleAndKeywordsFast(
- args, kwds, &_parser, &name, &BPyGPUTexture_Type, &py_texture, &py_samplerstate)) {
+ if (!PyArg_ParseTuple(
+ args, "sO!:GPUShader.uniform_sampler", &name, &BPyGPUTexture_Type, &py_texture)) {
return NULL;
}
- int sampler_state = GPU_SAMPLER_DEFAULT;
- if (py_samplerstate) {
- if (PyC_FlagSet_ToBitfield(pygpu_texture_samplerstate_items,
- py_samplerstate,
- &sampler_state,
- "shader.uniform_sampler") == -1) {
- return NULL;
- }
- }
-
int slot = GPU_shader_get_texture_binding(self->shader, name);
- GPU_texture_bind_ex(py_texture->tex, (eGPUSamplerState)sampler_state, slot, false);
+ GPU_texture_bind(py_texture->tex, slot);
GPU_shader_uniform_1i(self->shader, name, slot);
Py_RETURN_NONE;
@@ -665,7 +622,7 @@ static struct PyMethodDef pygpu_shader__tp_methods[] = {
pygpu_shader_uniform_int_doc},
{"uniform_sampler",
(PyCFunction)pygpu_shader_uniform_sampler,
- METH_VARARGS | METH_KEYWORDS,
+ METH_VARARGS,
pygpu_shader_uniform_sampler_doc},
{"uniform_block",
(PyCFunction)pygpu_shader_uniform_block,
More information about the Bf-blender-cvs
mailing list