0031317: Foundation Classes - confusing overloads of method gp_XY::SetCoord()

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0031317

Open CASCADE

[OCCT] OCCT:Foundation Classes

public

2020-01-23 19:09

2020-09-25 20:35

Reporter

abv

Assigned To

abv

Priority

normal

Severity

minor

Status

new

Resolution

open

Platform

OS Version

Product Version

Target Version

[OCCT] 7.6.0*

Fixed in Version

Summary

0031317: Foundation Classes - confusing overloads of method gp_XY::SetCoord()

Description

Method gp_XY::SetCoord() has two overloads:

void SetCoord (const Standard_Integer Index, const Standard_Real Xi);
void SetCoord (const Standard_Real X, const Standard_Real Y);

This is quite confusing because the user aware of existence of the second method can call SetCoord (1, 0) assuming that the vector should be set to [1, 0], but instead this call will set X to 0.

Tags

No tags attached.

Test case number

Attached Files

Relationships

Notes
(0090036) BenjaminBihler (updater) 2020-01-24 09:40	This is one of the drawbacks of abbreviations. In our code here we would write "SetCoordinates" and "SetCoordinate" and there would be no confusion. :)

Issue History
Date Modified	Username	Field	Change
2020-01-23 19:09	abv	New Issue
2020-01-23 19:09	abv	Assigned To	=> abv
2020-01-24 09:40	BenjaminBihler	Note Added: 0090036
2020-09-25 20:35	abv	Target Version	7.5.0 => 7.6.0*