OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

glsl: remove some continue statements

Browse Source 
In some cases, I think loop code is easier to read without continue
statements.

Reviewed-by: Timothy Arceri <tarceri@itsqueeze.com>
This commit is contained in:
Brian Paul
2017-12-15 14:36:25 -07:00
parent 76fc24ba8d
commit 544f41ff19
1 changed files with 11 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/compiler/glsl/link_varyings.cpp
View File

@@ -1277,9 +1277,7 @@ parse_tfeedback_decls(struct gl_context *ctx, struct gl_shader_program *prog,
* feedback of arrays would be useless otherwise. * feedback of arrays would be useless otherwise.
*/ */
for (unsigned j = 0; j < i; ++j) { for (unsigned j = 0; j < i; ++j) {
if (!decls[j].is_varying()) if (decls[j].is_varying()) {
continue;
if (tfeedback_decl::is_same(decls[i], decls[j])) { if (tfeedback_decl::is_same(decls[i], decls[j])) {
linker_error(prog, "Transform feedback varying %s specified " linker_error(prog, "Transform feedback varying %s specified "
"more than once.", varying_names[i]); "more than once.", varying_names[i]);
@@ -1287,6 +1285,7 @@ parse_tfeedback_decls(struct gl_context *ctx, struct gl_shader_program *prog,
} }
} }
} }
}
return true; return true;
} }
@@ -2571,14 +2570,13 @@ assign_varying_locations(struct gl_context *ctx,
matches.store_locations(); matches.store_locations();
for (unsigned i = 0; i < num_tfeedback_decls; ++i) { for (unsigned i = 0; i < num_tfeedback_decls; ++i) {
if (!tfeedback_decls[i].is_varying()) if (tfeedback_decls[i].is_varying()) {
continue;
if (!tfeedback_decls[i].assign_location(ctx, prog)) { if (!tfeedback_decls[i].assign_location(ctx, prog)) {
_mesa_hash_table_destroy(tfeedback_candidates, NULL); _mesa_hash_table_destroy(tfeedback_candidates, NULL);
return false; return false;
} }
} }
}
_mesa_hash_table_destroy(tfeedback_candidates, NULL); _mesa_hash_table_destroy(tfeedback_candidates, NULL);
if (consumer && producer) { if (consumer && producer) {