1
0
Fork 0

cogl: Remove unused CoglPipelineEvalFlags

Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/3437>
This commit is contained in:
Bilal Elmoussaoui 2023-12-02 11:23:06 +01:00 committed by Marge Bot
parent 893e6e49b1
commit 5d9e9707dd
9 changed files with 16 additions and 42 deletions

View file

@ -446,8 +446,7 @@ compare_entry_pipelines (CoglJournalEntry *entry0, CoglJournalEntry *entry1)
entry1->pipeline,
(COGL_PIPELINE_STATE_ALL &
~COGL_PIPELINE_STATE_COLOR),
COGL_PIPELINE_LAYER_STATE_ALL,
0))
COGL_PIPELINE_LAYER_STATE_ALL))
return TRUE;
else
return FALSE;
@ -1883,8 +1882,7 @@ _cogl_journal_try_read_pixel (CoglJournal *journal,
if (!_cogl_pipeline_equal (ctx->opaque_color_pipeline, entry->pipeline,
(COGL_PIPELINE_STATE_ALL &
~COGL_PIPELINE_STATE_COLOR),
COGL_PIPELINE_LAYER_STATE_ALL,
0))
COGL_PIPELINE_LAYER_STATE_ALL))
return FALSE;

View file

@ -88,8 +88,7 @@ entry_equal (const void *a,
return _cogl_pipeline_equal (entry_a->parent.pipeline,
entry_b->parent.pipeline,
hash->main_state,
hash->layer_state,
0);
hash->layer_state);
}
void
@ -185,8 +184,7 @@ _cogl_pipeline_hash_table_get (CoglPipelineHashTable *hash,
dummy_entry.hash = hash;
dummy_entry.hash_value = _cogl_pipeline_hash (key_pipeline,
hash->main_state,
hash->layer_state,
0);
hash->layer_state);
entry = g_hash_table_lookup (hash->table, &dummy_entry);
if (entry)

View file

@ -296,8 +296,7 @@ _cogl_pipeline_layer_resolve_authorities (CoglPipelineLayer *layer,
gboolean
_cogl_pipeline_layer_equal (CoglPipelineLayer *layer0,
CoglPipelineLayer *layer1,
unsigned long differences_mask,
CoglPipelineEvalFlags flags);
unsigned long differences_mask);
CoglPipelineLayer *
_cogl_pipeline_layer_pre_change_notify (CoglPipeline *required_owner,

View file

@ -43,8 +43,7 @@ _cogl_pipeline_set_layer_unit (CoglPipeline *required_owner,
gboolean
_cogl_pipeline_layer_texture_data_equal (CoglPipelineLayer *authority0,
CoglPipelineLayer *authority1,
CoglPipelineEvalFlags flags);
CoglPipelineLayer *authority1);
gboolean
_cogl_pipeline_layer_combine_state_equal (CoglPipelineLayer *authority0,

View file

@ -691,8 +691,7 @@ cogl_pipeline_add_layer_snippet (CoglPipeline *pipeline,
gboolean
_cogl_pipeline_layer_texture_data_equal (CoglPipelineLayer *authority0,
CoglPipelineLayer *authority1,
CoglPipelineEvalFlags flags)
CoglPipelineLayer *authority1)
{
if (authority0->texture == NULL)
{

View file

@ -636,8 +636,7 @@ _cogl_pipeline_layer_resolve_authorities (CoglPipelineLayer *layer,
gboolean
_cogl_pipeline_layer_equal (CoglPipelineLayer *layer0,
CoglPipelineLayer *layer1,
unsigned long differences_mask,
CoglPipelineEvalFlags flags)
unsigned long differences_mask)
{
unsigned long layers_difference;
CoglPipelineLayer *authorities0[COGL_PIPELINE_LAYER_STATE_SPARSE_COUNT];
@ -664,8 +663,7 @@ _cogl_pipeline_layer_equal (CoglPipelineLayer *layer0,
CoglPipelineLayerStateIndex state_index =
COGL_PIPELINE_LAYER_STATE_TEXTURE_DATA_INDEX;
if (!_cogl_pipeline_layer_texture_data_equal (authorities0[state_index],
authorities1[state_index],
flags))
authorities1[state_index]))
return FALSE;
}

View file

@ -228,7 +228,6 @@ typedef struct
typedef struct _CoglPipelineHashState
{
unsigned long layer_differences;
CoglPipelineEvalFlags flags;
unsigned int hash;
} CoglPipelineHashState;
@ -700,14 +699,12 @@ gboolean
_cogl_pipeline_equal (CoglPipeline *pipeline0,
CoglPipeline *pipeline1,
unsigned int differences,
unsigned long layer_differences,
CoglPipelineEvalFlags flags);
unsigned long layer_differences);
unsigned int
_cogl_pipeline_hash (CoglPipeline *pipeline,
unsigned int differences,
unsigned long layer_differences,
CoglPipelineEvalFlags flags);
unsigned long layer_differences);
/* Makes a copy of the given pipeline that is a child of the root
* pipeline rather than a child of the source pipeline. That way the

View file

@ -1846,8 +1846,7 @@ _cogl_pipeline_apply_overrides (CoglPipeline *pipeline,
static gboolean
_cogl_pipeline_layers_equal (CoglPipeline *authority0,
CoglPipeline *authority1,
unsigned long differences,
CoglPipelineEvalFlags flags)
unsigned long differences)
{
int i;
@ -1861,8 +1860,7 @@ _cogl_pipeline_layers_equal (CoglPipeline *authority0,
{
if (!_cogl_pipeline_layer_equal (authority0->layers_cache[i],
authority1->layers_cache[i],
differences,
flags))
differences))
return FALSE;
}
return TRUE;
@ -2008,8 +2006,7 @@ gboolean
_cogl_pipeline_equal (CoglPipeline *pipeline0,
CoglPipeline *pipeline1,
unsigned int differences,
unsigned long layer_differences,
CoglPipelineEvalFlags flags)
unsigned long layer_differences)
{
unsigned long pipelines_difference;
CoglPipeline *authorities0[COGL_PIPELINE_STATE_SPARSE_COUNT];
@ -2140,8 +2137,7 @@ _cogl_pipeline_equal (CoglPipeline *pipeline0,
{
if (!_cogl_pipeline_layers_equal (authorities0[bit],
authorities1[bit],
layer_differences,
flags))
layer_differences))
goto done;
break;
}
@ -2492,8 +2488,7 @@ _cogl_pipeline_init_state_hash_functions (void)
unsigned int
_cogl_pipeline_hash (CoglPipeline *pipeline,
unsigned int differences,
unsigned long layer_differences,
CoglPipelineEvalFlags flags)
unsigned long layer_differences)
{
CoglPipeline *authorities[COGL_PIPELINE_STATE_SPARSE_COUNT];
unsigned int mask;
@ -2503,7 +2498,6 @@ _cogl_pipeline_hash (CoglPipeline *pipeline,
state.hash = 0;
state.layer_differences = layer_differences;
state.flags = flags;
_cogl_pipeline_update_real_blend_enable (pipeline, FALSE);

View file

@ -72,14 +72,6 @@ typedef enum
COGL_N_PRIVATE_FEATURES
} CoglPrivateFeature;
/* Sometimes when evaluating pipelines, either during comparisons or
* if calculating a hash value we need to tweak the evaluation
* semantics */
typedef enum _CoglPipelineEvalFlags
{
COGL_PIPELINE_EVAL_FLAG_NONE = 0
} CoglPipelineEvalFlags;
void
_cogl_transform_point (const graphene_matrix_t *matrix_mv,
const graphene_matrix_t *matrix_p,