Displaying 2 results from an estimated 2 matches for "tonext".
Did you mean:
toext
2009 Jun 27
0
[LLVMdev] Patch for llvm::DepthFirstIterator.h and llvm::PostOrderIterator.h
...This patch looks good to me. I just have a few minor comments:
> + inline df_iterator() { CurrentTopNode = 0; /* End is when stack
is empty */ }
Should the comment here be updated to say that the End
is reached when the stack is empty and when CurrentTopNode
is null?
> + inline void toNext()
> + {
LLVM style puts the open brace on the same line as the function name.
> + inline _Self& skipChilds() { // skips all childs of the current
node and traverses to next node
The plural of "child" is "children"; please rename this function
accordingly. Al...
2009 Jun 26
3
[LLVMdev] Patch for llvm::DepthFirstIterator.h and llvm::PostOrderIterator.h
Hi @clang and @llvm,
attached you'll find a patch dealing with some iterator issues I already
mentioned in both lists. Since there was no reaction I cross-post again
- now IMHO production-ready code.
The patch is considered to get checked-in out of the box. It should not
affect the behavior of existing and working code. I really need it for
clang AST processing.
Changes:
1. Both