c++ array of iterators

Question

Why does this code executed in debug mode trigger a breakpoint?

#include <list>

void main() {

  std::list<int>::iterator* iterators = new std::list<int>::iterator[50];

  delete iterators;

}

*Why does this code executed in debug mode trigger a breakpoint?* -- Ok, I have to ask the question -- does your real code actually do things like this? — PaulMcKenzie, Jan 24 '18 at 14:24
Forgive me the 'void main' it was just to write a code snippet, thanks for the answers. I have to store some linked list element pointers, do you have a better way to do it? I know that they might be invalidated by some operations but I need to know given an element where it is placed in the list. — acco93, Jan 24 '18 at 14:25
@acco93 *I have to store some linked list element pointers, do you have a better way to do it* -- `std::vector::iterator> iterators(50);` Then there is no need for `delete []`, — PaulMcKenzie, Jan 24 '18 at 14:28

score 3 · Accepted Answer · edited Jan 24 '18 at 14:32

3

As said in comments, if you instantiate an array using

... = new name[];

you must use

delete [] name;

edited Jan 24 '18 at 14:32

Gaurav Sehgal

answered Jan 24 '18 at 14:24

Benjamin Barrois

score 2 · Answer 2 · answered Jan 24 '18 at 14:26

2

When you use the operator new[] you must also use the operator delete[], otherwise it is an undefined behavior.

You can check this question for more information: Is delete[] equal to delete?

answered Jan 24 '18 at 14:26

Sebi

2 Answers2