IfcPolygonialFaceSet constructor generates wrong empty list

[seb-esser]

Seems like the same issue I was reporting in #31 :

#33= IFCPOLYGONALFACESET(#32,$,(#),$);

@jmirtsch would you mind spending a look on it? I was using: var ptList = new IfcCartesianPointList3D(database, new List<Tuple<double, double, double>>()); hostProduct.Representation = new IfcProductDefinitionShape( new IfcShapeRepresentation( new IfcPolygonalFaceSet( ptList, new List<IfcIndexedPolygonalFace>()))); , which has produced the given output.

Thanks!

[jmirtsch]

Hi Sebastian,

That output isn't right, but unfortunately neither is

33= IFCPOLYGONALFACESET(#32,$,(),$);

The attribute definition is this: Faces : LIST [1:?] OF IfcIndexedPolygonalFace; requiring the list to have at least one element.

I don't necessarily think the constructor should prevent an empty list to be passed, but it's up to the code calling it to add at least one face to the instance.

Let me know if this doesn't make sense.

Cheers,

Jon

[seb-esser]

fair enough. I thought, maybe a $ might suit better if no face is given. But I agree with your argumentation of LIST [1,:], so lets stick to the existing solution