search

aleuly / ifcplusplus

Automatically exported from code.google.com/p/ifcplusplus
Other
0 stars 0 forks source link

SI unit length metric without prefix should be valid #34

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
SI unit for metric length should be valid even if there is no "IfcSIPrefix" as 
it is optional. If it is not there then unit scale factor of 1.0 should be 
valid.

In UnitConvrter.cpp in function 
void UnitConverter::setIfcProject(shared_ptr<IfcProject> project)()
{
..
..
..
if( unit_type->m_enum == IfcUnitEnum::ENUM_LENGTHUNIT )
{
    if( si_unit->m_Prefix )
   {
    ...
    ...
     length_factor_found = true;  // Move this outside of this scope
   }

  length_factor_found = true;  // Should be here
}
..
..
..
}

..
..
..
}

Original issue reported on code.google.com by sajlo...@gmail.com on 8 Jan 2015 at 3:58

GoogleCodeExporter commented 9 years ago 
Yes, I agree. I will check that in soon.
Thanks :)

Original comment by fabian.g...@gmail.com on 8 Jan 2015 at 11:17