ifcplusplus icon indicating copy to clipboard operation
ifcplusplus copied to clipboard

SI unit length metric without prefix should be valid

Open GoogleCodeExporter opened this issue 10 years ago • 1 comments

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 [email protected] on 8 Jan 2015 at 3:58

GoogleCodeExporter avatar Apr 01 '15 13:04 GoogleCodeExporter

Yes, I agree. I will check that in soon.
Thanks :)

Original comment by [email protected] on 8 Jan 2015 at 11:17

  • Changed state: Started
  • Added labels: ****
  • Removed labels: ****

GoogleCodeExporter avatar Apr 01 '15 13:04 GoogleCodeExporter