[scfinder] processing input origins

[FMassin]

Adds origin input to scfinder.
Default delay warning threshold changed from 3s to 5s.

[FMassin]

Use my own separate local FinDer() object directly with .process() (it's a pointer)

[FMassin]

Try to make a separate local .copy() and append to this one instead (make sure it's a deep copy).

[FMassin]

Remove

[FMassin]

@jenrandrews: is this the best way to copy the _latestMaxPGAs data ?

[jenrandrews]

Hi Fred, since PGA_Data_List objects are just vectors containing objects of PGA_Data class, I believe you can just do A = B, and A will be a deep copy of B. This may (or may not) be more efficient than your explicit copy loop, since it may not involve vector resizing. However, there is no copy constructor defined for the PGA_Data class, so something may go awry. My suggestion would be to do a quick test: try the originBasedLatestMaxPGAs = _latestMaxPGAs method, change something in one of the PGA_Data objects, and see if it shows up in the other list. If it doesn't, you're golden, otherwise keep what you have.

[FMassin]

@jenrandrews @ sceylan : is it correct that originBasedFinder and originBasedLatestMaxPGAs will be destroyed once this method exits?

[jenrandrews]

I took a look at the full file and I think originBasedLatestMaxPGAs is truly locally scoped and will be destroyed. For originBasedFinder you are creating it with a "new" which means you should also call "delete" when done. So just add the line "delete originBasedFinder;" whenever in the function/code you are finished with the object (looks like line 647). The object looks locally scoped though, so you could also just create it with "Finder originBasedFinder = Finder(...);" and switch to "originBasedFinder.process()" and "sendFinder(&originBasedFinder)". Either approach should work =)