Review: to fix issue #1979

[msramalho]

Implements an iterator for any CtElement that can be used as follows:

CtIterator iterator = new CtIterator(/** some CtElement*/); 
while (iterator.hasNext()) { 
    CtElement next = (CtElement) iterator.next(); 
    System.out.println(next.toString()); 
}

[msramalho]

Need some help on:

• using checkstyle in intellij, already installed
• creating the tests

this refers to issue #1979

[monperrus]

Thanks a lot for this PR!

creating the tests

Here is a suggestion. One can visit of a small tree, put the elements in a list, a verify the order of the list in an assertion.

[msramalho]

I think this is ready for merge, if you want someone who knows how to improve the tests by using an AbstractProcessor wants to do so, they are welcome 😄

[monperrus]

Looks like the PR for #1077. Correct?

[msramalho]

Actually the PR for #1979

A friend of mine may actually suggested that this CtIterator can be used to fix #1077 as well, just by wrapping it and providing a class that implements Iterable. He might have something to say @AndreFCruz

[monperrus]

I'm OK to merge this one: good feature, good test, good documentation.

[monperrus]

We'll discuss about #1077 after that. @tdurieux suggests a simple iterator method in CtElement.

[monperrus]

Thanks a lot for this first contribution, looking forward to the one for #1077 @AndreFCruz