ESCOMP / ESCOMP/CAM

Bugzilla 1972: add_hist_coord - look into misleading overloading

Open
#2 0 comments 0 reactions 1 assignee Claimed by @gold2718 View on GitHub
bug
Dominant language
No language data
Stars
91
Forks
183
Avg merge
6d 2m
Merged PRs (30d)
9

Description

Opened: Cheryl Craig 2014-05-08 14:12:48 MDT
add_hist_coord has 3 overloaded specific routines.

The actual minimal user interface is (name, value, label). If a user neglects to specify 3 values and only gives two (name, value), one of the overloaded routines (add_hist_coord_regonly) is called and value is overwritten with index.

This is misleading and should probably look at whether add_hist_coord_regonly should be part of the add_hist_coord overloading or not.

Reviewed: Cheryl Craig 2016-05-04 14:12:07 MDT
This might go away with the refactor.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.