XMLWordPrintable

Details

• Type: Improvement
• Status: Invalid
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
• Story Points:
2

Description

Currently, to add multiple new empty records to a sourceCatalog requires multiple calls to catalog.addNew(), which can result in the catalog being non-contiguous in memory. It would be much better to have an catalog.addNew(size_t N) method that creates N new elements at once, all contiguous.

Whether this is best done via an overload of addNew or a new method, I leave to the implementer.

Activity

No builds found.
John Parejko created issue -
Hide
Paul Price added a comment - - edited

Workaround:

 catalog.reserve(num) for ii in range(num):  catalog.addNew() 

Show
Paul Price added a comment - - edited Workaround: catalog.reserve(num) for ii in range(num): catalog.addNew()
Field Original Value New Value
Hide
Tim Jenness added a comment -

What do we want to do with this old ticket? Is the workaround from Paul Price sufficient?

Show
Tim Jenness added a comment - What do we want to do with this old ticket? Is the workaround from Paul Price sufficient?
Hide
Eli Rykoff added a comment -

This has been fixed at some point, there was a resize method which does this exact operation. How do we mark this ticket then?

Show
Eli Rykoff added a comment - This has been fixed at some point, there was a resize method which does this exact operation. How do we mark this ticket then?
Hide
Eli Rykoff added a comment -

Show
 Link This issue duplicates DM-19461 [ DM-19461 ]
 Resolution Done [ 10000 ] Status To Do [ 10001 ] Invalid [ 11005 ]

People

Assignee:
Unassigned
Reporter:
John Parejko
Watchers:
Eli Rykoff, Jim Bosch, John Parejko, John Swinbank, Paul Price, Russell Owen, Tim Jenness