Attention is currently required from: Nico Huber, Edward O'Callaghan, Daniel Campello, Anastasia Klimchuk.

View Change

2 comments:

• File layout.c:

  • Patch Set #4, Line 369: return iterator ? iterator->next : layout->head;

    This duplicates _layout_next. Should one of them be removed?

  • Patch Set #4, Line 418: entry->next = layout->head;

    Shouldn't this be NULL? Otherwise it's creating a reference cycle.

To view, visit change 33521. To unsubscribe, or for help writing mail filters, visit settings.