File: StepToTopoDS_TranslateFace.cxx Lines: 422, 466 Problem: aFS variable is never initialized, so condition (!aFS.IsNull()) is always true. Lines: 459-362 Problem: It would be more correct if we calculate the missing normals Files: StepToTopoDS_TranslateSolid.cxx StepToTopoDS_TranslateShellSolid.cxx StepToTopoDS_TranslateFace.cxx Problem: Suppose that the STEP file contains only tessellated representation of the model. Where is the protection for double construction of the TopoDS_Shell and triple construction of the TopoDS_Face during proceeding of the StepVisual_TessellatedSolid, StepVisual_TessellatedShell, StepVisual_TessellatedFace? File: TopoDSToStep_MakeManifoldSolidBrep.cxx Lines: 88, 113 Problem: Done flag is true if at least one of the results is not null. Is it normal? There is no father protection from the null result.