Bugs item #3441645, was opened at 2011-11-23 20:21
Message generated for change (Tracker Item Submitted) made by cpudney
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=744692&aid=3441645&group_id=139835

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Data processing
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Chris Pudney (cpudney)
Assigned to: Nobody/Anonymous (nobody)
Summary: SimpleChromatographicPeak constructor creates references

Initial Comment:
The SimpleChromatographicPeak(ChromatographicPeak) is designed to create a new 
copy of a peak.  In many cases it creates references to the properties of the 
original peak it is copying.  This means that if these properties are changed 
in the original peak, a side-effect is that they are also changed in the copy, 
e.g.

ChromatographicPeak copy = new SimpleChromatographicPeak(orig);    // Create 
copy.
copy.getRawDataPointsRTRange().extendRange(1000)                              
// Also modifies the RT range of orig!!!

SimpleChromatographicPeak(ChromatographicPeak) should create its own instances 
of any mutable properties.

Similar problems exist in GaussianPeak and ResolvedPeak.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=744692&aid=3441645&group_id=139835

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and more. Splunk takes this 
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
Mzmine-devel mailing list
Mzmine-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mzmine-devel

Reply via email to