3DPrinting/PrusaSlicer-NonPlainar
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix bad deserialization of extruder_offset = 0x0

Browse Source
This commit is contained in:
Alessandro Ranellucci 2014-03-24 14:16:37 +01:00
parent 7a58457add
commit 4c6f9703df
3 changed files with 33 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
xs/src/Config.cpp
View File

@ -22,7 +22,8 @@ ConfigBase::apply(const ConfigBase &other, bool ignore_nonexistent) {
} }
// not the most efficient way, but easier than casting pointers to subclasses // not the most efficient way, but easier than casting pointers to subclasses
my_opt->deserialize( other.option(*it)->serialize() ); bool res = my_opt->deserialize( other.option(*it)->serialize() );
if (!res) CONFESS("Unexpected failure when deserializing serialized value");
} }
} }

27
xs/src/Config.hpp
View File

@ -265,8 +265,14 @@ class ConfigOptionPoint : public ConfigOption
}; };
bool deserialize(std::string str) { bool deserialize(std::string str) {
int res = sscanf(str.c_str(), "%lf%*1[,x]%lf", &this->point.x, &this->point.y); if (strncmp(str.c_str(), "0x", 2) == 0) {
return res == 2; this->point.x = 0;
int res = sscanf(str.c_str()+2, "%lf", &this->point.y);
return res == 1;
} else {
int res = sscanf(str.c_str(), "%lf%*1[,x]%lf", &this->point.x, &this->point.y);
return res == 2;
}
}; };
}; };
@ -286,15 +292,24 @@ class ConfigOptionPoints : public ConfigOption, public ConfigOptionVector<Pointf
}; };
bool deserialize(std::string str) { bool deserialize(std::string str) {
this->values.clear(); std::vector<Pointf> values;
std::istringstream is(str); std::istringstream is(str);
std::string point_str; std::string point_str;
while (std::getline(is, point_str, ',')) { while (std::getline(is, point_str, ',')) {
Pointf point; Pointf point;
int res = sscanf(point_str.c_str(), "%lfx%lf", &point.x, &point.y); if (strncmp(point_str.c_str(), "0x", 2) == 0) {
if (res != 2) return false; // if string starts with "0x", only apply sscanf() to the second coordinate
this->values.push_back(point); // otherwise it would parse the string as a hex number
point.x = 0;
int res = sscanf(point_str.c_str()+2, "%lf", &point.y);
if (res != 1) return false;
} else {
int res = sscanf(point_str.c_str(), "%lfx%lf", &point.x, &point.y);
if (res != 2) return false;
}
values.push_back(point);
} }
this->values = values;
return true; return true;
}; };
}; };

11
xs/t/15_config.t
View File

@ -4,7 +4,7 @@ use strict;
use warnings; use warnings;
use Slic3r::XS; use Slic3r::XS;
use Test::More tests => 94; use Test::More tests => 95;
foreach my $config (Slic3r::Config->new, Slic3r::Config::Full->new) { foreach my $config (Slic3r::Config->new, Slic3r::Config::Full->new) {
$config->set('layer_height', 0.3); $config->set('layer_height', 0.3);
@ -144,4 +144,13 @@ foreach my $config (Slic3r::Config->new, Slic3r::Config::Full->new) {
} }
} }
{
my $config = Slic3r::Config->new;
# the pair [0,0] is part of the test, since it checks whether the 0x0 serialized value is correctly parsed
$config->set('extruder_offset', [ [0,0], [20,0], [0,20] ]);
my $config2 = Slic3r::Config->new;
$config2->apply($config);
is_deeply $config->get('extruder_offset'), $config2->get('extruder_offset'), 'apply dynamic over dynamic';
}
__END__ __END__