Reapply "Fix (unreported) Assets: MEM_new/MEM_freeN mismatch usages."
This reverts commit cb76781be74320e2883b66de053dcb97e79ea169, and fixes the issues from original da05bff96c2209 commit (missing initialization of a pointer in copy constructor in some cases, and forgot to handle one allocated string in the move constructor). Many thanks to @julianeisel for finding the actual issues here.
This commit is contained in:
parent
6320066a88
commit
6aa6aee2d5
@ -7,6 +7,7 @@
|
||||
*/
|
||||
|
||||
#include <cstring>
|
||||
#include <utility>
|
||||
|
||||
#include "DNA_ID.h"
|
||||
#include "DNA_defaults.h"
|
||||
@ -42,26 +43,44 @@ void BKE_asset_metadata_free(AssetMetaData **asset_data)
|
||||
|
||||
AssetMetaData *BKE_asset_metadata_copy(const AssetMetaData *source)
|
||||
{
|
||||
AssetMetaData *copy = BKE_asset_metadata_create();
|
||||
|
||||
copy->local_type_info = source->local_type_info;
|
||||
|
||||
if (source->properties) {
|
||||
copy->properties = IDP_CopyProperty(source->properties);
|
||||
return MEM_new<AssetMetaData>(__func__, *source);
|
||||
}
|
||||
|
||||
BKE_asset_metadata_catalog_id_set(copy, source->catalog_id, source->catalog_simple_name);
|
||||
AssetMetaData::AssetMetaData(const AssetMetaData &other)
|
||||
: local_type_info(other.local_type_info),
|
||||
properties(nullptr),
|
||||
catalog_id(other.catalog_id),
|
||||
active_tag(other.active_tag),
|
||||
tot_tags(other.tot_tags)
|
||||
{
|
||||
if (other.properties) {
|
||||
properties = IDP_CopyProperty(other.properties);
|
||||
}
|
||||
|
||||
copy->author = BLI_strdup_null(source->author);
|
||||
copy->description = BLI_strdup_null(source->description);
|
||||
copy->copyright = BLI_strdup_null(source->copyright);
|
||||
copy->license = BLI_strdup_null(source->license);
|
||||
STRNCPY(catalog_simple_name, other.catalog_simple_name);
|
||||
|
||||
BLI_duplicatelist(©->tags, &source->tags);
|
||||
copy->active_tag = source->active_tag;
|
||||
copy->tot_tags = source->tot_tags;
|
||||
author = BLI_strdup_null(other.author);
|
||||
description = BLI_strdup_null(other.description);
|
||||
copyright = BLI_strdup_null(other.copyright);
|
||||
license = BLI_strdup_null(other.license);
|
||||
|
||||
return copy;
|
||||
BLI_duplicatelist(&tags, &other.tags);
|
||||
}
|
||||
|
||||
AssetMetaData::AssetMetaData(AssetMetaData &&other)
|
||||
: local_type_info(other.local_type_info),
|
||||
properties(std::exchange(other.properties, nullptr)),
|
||||
catalog_id(other.catalog_id),
|
||||
author(std::exchange(other.author, nullptr)),
|
||||
description(std::exchange(other.description, nullptr)),
|
||||
copyright(std::exchange(other.copyright, nullptr)),
|
||||
license(std::exchange(other.license, nullptr)),
|
||||
active_tag(other.active_tag),
|
||||
tot_tags(other.tot_tags)
|
||||
{
|
||||
STRNCPY(catalog_simple_name, other.catalog_simple_name);
|
||||
tags = other.tags;
|
||||
BLI_listbase_clear(&other.tags);
|
||||
}
|
||||
|
||||
AssetMetaData::~AssetMetaData()
|
||||
|
@ -3210,8 +3210,9 @@ static void filelist_readjob_list_lib_add_datablock(FileListReadJob *job_params,
|
||||
if (job_params->load_asset_library) {
|
||||
/* Take ownership over the asset data (shallow copies into unique_ptr managed memory) to
|
||||
* pass it on to the asset system. */
|
||||
std::unique_ptr metadata = std::make_unique<AssetMetaData>(*datablock_info->asset_data);
|
||||
MEM_freeN(datablock_info->asset_data);
|
||||
std::unique_ptr metadata = std::make_unique<AssetMetaData>(
|
||||
std::move(*datablock_info->asset_data));
|
||||
MEM_delete(datablock_info->asset_data);
|
||||
/* Give back a non-owning pointer, because the data-block info is still needed (e.g. to
|
||||
* update the asset index). */
|
||||
datablock_info->asset_data = metadata.get();
|
||||
|
@ -84,6 +84,9 @@ typedef struct AssetMetaData {
|
||||
char _pad[4];
|
||||
|
||||
#ifdef __cplusplus
|
||||
AssetMetaData() = default;
|
||||
AssetMetaData(const AssetMetaData &other);
|
||||
AssetMetaData(AssetMetaData &&other);
|
||||
/** Enables use with `std::unique_ptr<AssetMetaData>`. */
|
||||
~AssetMetaData();
|
||||
#endif
|
||||
|
Loading…
Reference in New Issue
Block a user