|Anonymous | Login||2022-01-22 21:48 CST|
|My View | View Issues|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0000137||Rosetta||[All Projects] Bad Coding||public||2012-10-18 16:37||2013-02-26 10:58|
|Fixed in Version|
|Summary||0000137: Why does core::kinematics::Edge keep start and stop as int instead of core::Size?|
|Description||Why does core::kinematics::Edge keep start and stop as int instead of core::Size? start and stop refer to residue indices in a Pose, which can't be negative. Holdover or is there a good reason? I bet it's related to a lot of warnings.|
|Tags||No tags attached.|
|Command Line Used||--|
|Fixed in SVN Version|
It's not just Edge; a similar problem is in FoldTree. ...And in lots of other places.
As I'm going through the code removing compiler warnings, this (int instead of core::Size) issue comes up probably more than any other. It's sad how little of the code follows the coding conventions.
Anyhow, since we've cleaned out most of the warnings from core, refactoring Edge/FoldTree, etc., would likely uproot a whole ton more warnings. Might I ask that no one tries to do this until after we get a warning-free build test up?
As a side note, do you think we should post a "Bad Coding" bug report for every blatant disregard of the conventions? Because I could start a large number of bug reports, then, as we've been going through the code from the bottom up and have the pleasure of observing all sorts of fun. It could perhaps be useful, if we really do care to clean up the community's code base and want to track progress; or else it could be a little extreme. What do you think?
This code predates the coding conventions. (Most of core does).
This is a bug, not just a coding convention violation, because it allows nonsense arguments. Edge's start and stop ultimately refer to resid indices, which have to be positive, so it is flatly incorrect for them to ever have negative values as far as I understand.
I think you can and should report a bug for every little thing that annoys you. We altered the system to not email bugs except if users want to be on the bug tracker's list, or they are actually involved in the bug, so it's not like flooding the tracker is a problem. That said, for minor convention violations (especially purely stylistic like indentation), it'll cost you more time to report than just fix and move on.
|At Minicon2013, official permission to fix this was granted|
|2012-10-18 16:37||smlewis||New Issue|
|2012-10-19 13:58||Labonte||Note Added: 0000121|
|2012-10-19 13:58||Labonte||Assigned To||=> Labonte|
|2012-10-19 13:58||Labonte||Status||new => confirmed|
|2012-10-19 13:59||Labonte||Assigned To||Labonte =>|
|2012-10-19 14:52||smlewis||Note Added: 0000122|
|2012-12-09 14:52||Labonte||Relationship added||related to 0000159|
|2013-02-25 14:29||smlewis||Note Added: 0000143|
|2013-02-26 10:58||Labonte||Assigned To||=> Labonte|
|2013-02-26 10:58||Labonte||Status||confirmed => assigned|
|Copyright © 2000 - 2012 MantisBT Group|