griefith/test
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Alembic export: avoid create-and-reset of shared pointer

Browse Source 
Constructing the shared pointer where the object is actually allocated
makes the code a bit clearer.
This commit is contained in:
Sybren A. Stüvel
2017-05-26 12:56:27 +02:00
parent cfce8623a8
commit 4aeba3b90d
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
source/blender/alembic/intern/abc_exporter.cc
View File

@@ -219,20 +219,17 @@ void AbcExporter::getShutterSamples(double step, bool time_relative,
Alembic::Abc::TimeSamplingPtr AbcExporter::createTimeSampling(double step)
{
TimeSamplingPtr time_sampling;
std::vector<double> samples;
if (m_settings.frame_start == m_settings.frame_end) {
time_sampling.reset(new Alembic::Abc::TimeSampling());
return time_sampling;
return TimeSamplingPtr(new Alembic::Abc::TimeSampling());
}
getShutterSamples(step, true, samples);
Alembic::Abc::TimeSamplingType ts(static_cast<uint32_t>(samples.size()), 1.0 / m_scene->r.frs_sec);
time_sampling.reset(new Alembic::Abc::TimeSampling(ts, samples));
return time_sampling;
return TimeSamplingPtr(new Alembic::Abc::TimeSampling(ts, samples));
}
void AbcExporter::getFrameSet(double step, std::set<double> &frames)