Stefan Xenos <sxe...@google.com> writes:

>> saw a few comments that say "needs further work".
>
> Could you clarify where you saw those comments? Were they something I
> wrote ...

Yeah, e.g.

       /* TODO: This causes a crash. It sets one of the atom_value handlers to
        * something invalid, which causes a crash later when we all
        * show_ref_array_item. Figure out why this happens and put back the 
sorting.
        *ref_array_sort(sorting, &array); */

was what caught my attention in the previous and this version.  As I
said, it is OK if the code is not completely finished.  It's an
iterative process.

By the way, our multi-line comments have opening /* and closing */
on their own lines.

Thanks.

Reply via email to