callisto-debian/gimp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

plug-ins: more refactoring of metadata-editor.

Browse Source 
There was a lot of duplicate code based on
whether or not value was NULL with for
some tags partially unreachable code.

We combine the parts and only test
value for NULL where needed.

Also rename the typos in enum
COL_REGSITRY_* to
COL_REGISTRY_*
This commit is contained in:
Jacob Boerema
2021-03-09 22:18:04 -05:00
parent e24a6dc8d6
commit 0a0eeac1ad
3 changed files with 1125 additions and 1697 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2804
plug-ins/metadata/metadata-editor.c
View File

File diff suppressed because it is too large Load Diff

6
plug-ins/metadata/metadata-tags.h
View File

@ -76,9 +76,9 @@ enum
enum
{
COL_REGSITRY_ORG_ID = 0,
COL_REGSITRY_ITEM_ID,
COL_REGSITRY_NUM_COLS
COL_REGISTRY_ORG_ID = 0,
COL_REGISTRY_ITEM_ID,
COL_REGISTRY_NUM_COLS
};
enum

12
plug-ins/metadata/metadata-xml.c
View File

@ -340,8 +340,8 @@ set_tag_ui (metadata_editor *args,
{
gtk_list_store_append (liststore, &iter);
gtk_list_store_set (liststore, &iter,
COL_REGSITRY_ORG_ID, list_tag_data[row][1],
COL_REGSITRY_ITEM_ID, list_tag_data[row][2],
COL_REGISTRY_ORG_ID, list_tag_data[row][1],
COL_REGISTRY_ITEM_ID, list_tag_data[row][2],
-1);
for (item = 1; item < n_registryid + 1; item++)
{
@ -359,8 +359,8 @@ set_tag_ui (metadata_editor *args,
{
gtk_list_store_append (liststore, &iter);
gtk_list_store_set (liststore, &iter,
COL_REGSITRY_ORG_ID, NULL,
COL_REGSITRY_ITEM_ID, NULL,
COL_REGISTRY_ORG_ID, NULL,
COL_REGISTRY_ITEM_ID, NULL,
-1);
}
}
@ -721,8 +721,8 @@ get_tag_ui_list (metadata_editor *args, gchar *name, gchar *mode)
else if (!strcmp (REGISTRYID_HEADER, name))
{
gtk_tree_model_get (treemodel, &iter,
COL_REGSITRY_ORG_ID, &tagdata[row][0],
COL_REGSITRY_ITEM_ID, &tagdata[row][1],
COL_REGISTRY_ORG_ID, &tagdata[row][0],
COL_REGISTRY_ITEM_ID, &tagdata[row][1],
-1);
if ((tagdata[row][0] != NULL && strlen(tagdata[row][0]) > 0) ||