0027373: GeomConvert_ApproxSurface missing documentation for argument PrecisCode. - MantisBT

MantisBT

Mantis Bug Tracker Workflow

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

ID

Project

Category

View Status

Date Submitted

Last Update

0027373

Community

[OCCT] OCCT:Modeling Data

public

2016-04-12 10:23

2017-12-05 17:09

Reporter

Vico Liang

Assigned To

msv

Priority

normal

Severity

minor

Status

assigned

Resolution

open

Platform

OS

OS Version

Product Version

[OCCT] 6.9.1

Target Version

[OCCT] 7.4.0*

Fixed in Version

Summary

0027373: GeomConvert_ApproxSurface missing documentation for argument PrecisCode.

Description

GeomConvert_ApproxSurface(const Handle(Geom_Surface)& Surf, const Standard_Real Tol3d, const GeomAbs_Shape UContinuity, const GeomAbs_Shape VContinuity, const Standard_Integer MaxDegU, const Standard_Integer MaxDegV, const Standard_Integer MaxSegments, const Standard_Integer PrecisCode);

Last argument PrecisCode missing detail comment, for a user, it should not know how to make it work.

I suggest to make it as an enum values.

Tags

No tags attached.

Test case number

Relationships

Notes
(0052598) msv (developer) 2016-04-12 12:09	Do you have an idea and would you like to implement it by your own? Then welcome to produce a branch with the fix.

(0052616) Vico Liang (developer) 2016-04-12 15:38	Dear msv, I find the comment from the internal implementation as below: //! PrecisCode : Precision on approximation's error measurement //! 1 : Fast computation and average precision //! 2 : Average computation and good precision //! 3 : Slow computation and very good precision It should be better to update it by OCCT team.

Issue History
Date Modified	Username	Field	Change
2016-04-12 10:23	Vico Liang	New Issue
2016-04-12 10:23	Vico Liang	Assigned To	=> msv
2016-04-12 12:09	msv	Note Added: 0052598
2016-04-12 12:09	msv	Assigned To	msv => Vico Liang
2016-04-12 12:09	msv	Status	new => feedback
2016-04-12 15:38	Vico Liang	Note Added: 0052616
2016-04-12 15:39	Vico Liang	Assigned To	Vico Liang => msv
2016-04-12 15:40	Vico Liang	Status	feedback => assigned
2016-10-25 15:53	msv	Target Version	7.1.0 => 7.2.0
2017-07-20 15:30	msv	Target Version	7.2.0 => 7.3.0
2017-12-05 17:09	msv	Target Version	7.3.0 => 7.4.0*

Copyright © 2000 - 2018 MantisBT Team