moving within a menu section with all disabled items hits an infinite loop

[dankamongmen]

If a menu section is created with entiretly disabled items, it can be opened, but then pressing up or down within that menu leads to the application locking up, pegging a cpu.

[dankamongmen]

heh, from ncmenu_previtem():

  // FIXME probably best to detect cycles 

indeed, nicholas, indeed!

[dankamongmen]

We now detect cycles in both ncmenu_previtem() and ncmenu_nextitem(), resolving this problem.

[dankamongmen]

i also now check for cycles in ncmenu_prevsection() and ncmenu_nextsection() as of f0976131e