[Bf-blender-cvs] [ff238e3] temp-array-modifier: Cleanup: naming
Campbell Barton
noreply at git.blender.org
Tue Aug 12 03:03:26 CEST 2014
Commit: ff238e3e668002368f14274ee3ec1e6a98480d85
Author: Campbell Barton
Date: Tue Aug 12 11:03:02 2014 +1000
Branches: temp-array-modifier
https://developer.blender.org/rBff238e3e668002368f14274ee3ec1e6a98480d85
Cleanup: naming
===================================================================
M source/blender/modifiers/intern/MOD_array.c
===================================================================
diff --git a/source/blender/modifiers/intern/MOD_array.c b/source/blender/modifiers/intern/MOD_array.c
index 0a573d0..ae38c6f 100644
--- a/source/blender/modifiers/intern/MOD_array.c
+++ b/source/blender/modifiers/intern/MOD_array.c
@@ -173,7 +173,7 @@ typedef struct SortVertsElem {
} SortVertsElem;
-static int vergaverco(const void *e1, const void *e2)
+static int svert_sum_cmp(const void *e1, const void *e2)
{
const SortVertsElem *sv1 = (SortVertsElem *)e1;
const SortVertsElem *sv2 = (SortVertsElem *)e2;
@@ -198,7 +198,7 @@ static void svert_from_mvert(SortVertsElem *sv, MVert *mv, int i_begin, int i_en
* It builds a mapping for all vertices within source, to vertices within target, or -1 if no double found
* The int doubles_map[num_verts_source] array must have been allocated by caller.
*/
-static void map_doubles(
+static void dm_mvert_map_doubles(
int *doubles_map,
MVert *mverts,
int target_start,
@@ -218,8 +218,8 @@ static void map_doubles(
source_end = source_start + source_num_verts;
/* build array of MVerts to be tested for merging */
- sorted_verts_target = MEM_callocN(sizeof(SortVertsElem) * target_num_verts, "MOD array map doubles 1");
- sorted_verts_source = MEM_callocN(sizeof(SortVertsElem) * source_num_verts, "MOD array map doubles 2");
+ sorted_verts_target = MEM_mallocN(sizeof(SortVertsElem) * target_num_verts, __func__);
+ sorted_verts_source = MEM_mallocN(sizeof(SortVertsElem) * source_num_verts, __func__);
/* Copy target vertices index and cos into SortVertsElem array */
svert_from_mvert(sorted_verts_target, mverts + target_start, target_start, target_end);
@@ -228,8 +228,8 @@ static void map_doubles(
svert_from_mvert(sorted_verts_source, mverts + source_start, source_start, source_end);
/* sort arrays according to sum of vertex coordinates (sumco) */
- qsort(sorted_verts_target, target_num_verts, sizeof(SortVertsElem), vergaverco);
- qsort(sorted_verts_source, source_num_verts, sizeof(SortVertsElem), vergaverco);
+ qsort(sorted_verts_target, target_num_verts, sizeof(SortVertsElem), svert_sum_cmp);
+ qsort(sorted_verts_source, source_num_verts, sizeof(SortVertsElem), svert_sum_cmp);
sve_target_low_bound = sorted_verts_target;
i_target_low_bound = 0;
@@ -581,14 +581,15 @@ static DerivedMesh *arrayModifier_doArray(
}
}
else {
- map_doubles(full_doubles_map,
- result_dm_verts,
- (c - 1) * chunk_nverts,
- chunk_nverts,
- c * chunk_nverts,
- chunk_nverts,
- amd->merge_dist,
- false);
+ dm_mvert_map_doubles(
+ full_doubles_map,
+ result_dm_verts,
+ (c - 1) * chunk_nverts,
+ chunk_nverts,
+ c * chunk_nverts,
+ chunk_nverts,
+ amd->merge_dist,
+ false);
}
}
}
@@ -603,14 +604,15 @@ static DerivedMesh *arrayModifier_doArray(
(count > 1))
{
/* Merge first and last copies */
- map_doubles(full_doubles_map,
- result_dm_verts,
- last_chunk_start,
- last_chunk_nverts,
- first_chunk_start,
- first_chunk_nverts,
- amd->merge_dist,
- false);
+ dm_mvert_map_doubles(
+ full_doubles_map,
+ result_dm_verts,
+ last_chunk_start,
+ last_chunk_nverts,
+ first_chunk_start,
+ first_chunk_nverts,
+ amd->merge_dist,
+ false);
}
/* start capping */
@@ -626,14 +628,15 @@ static DerivedMesh *arrayModifier_doArray(
start_cap_nverts, start_cap_nedges, start_cap_nloops, start_cap_npolys);
/* Identify doubles with first chunk */
if (amd->flags & MOD_ARR_MERGE) {
- map_doubles(full_doubles_map,
- result_dm_verts,
- first_chunk_start,
- first_chunk_nverts,
- start_cap_start,
- start_cap_nverts,
- amd->merge_dist,
- false);
+ dm_mvert_map_doubles(
+ full_doubles_map,
+ result_dm_verts,
+ first_chunk_start,
+ first_chunk_nverts,
+ start_cap_start,
+ start_cap_nverts,
+ amd->merge_dist,
+ false);
}
}
@@ -649,14 +652,15 @@ static DerivedMesh *arrayModifier_doArray(
end_cap_nverts, end_cap_nedges, end_cap_nloops, end_cap_npolys);
/* Identify doubles with last chunk */
if (amd->flags & MOD_ARR_MERGE) {
- map_doubles(full_doubles_map,
- result_dm_verts,
- last_chunk_start,
- last_chunk_nverts,
- end_cap_start,
- end_cap_nverts,
- amd->merge_dist,
- false);
+ dm_mvert_map_doubles(
+ full_doubles_map,
+ result_dm_verts,
+ last_chunk_start,
+ last_chunk_nverts,
+ end_cap_start,
+ end_cap_nverts,
+ amd->merge_dist,
+ false);
}
} /* Done capping */
More information about the Bf-blender-cvs
mailing list