Based on the surrounding code in that function, should there be an if (sd->main_layout) check before calling that?

This function can be expensive to call repeatedly...

Seems smart. Is it always the case that this object exists in a scrollable popup?

If this works 100% of the time that's fine, but generally you wouldn't check calculating if you're just setting the recalc flag; that function was added specifically so that you could know whether to set the flag or manually call recalc because if you are inside a calc then you want to trigger the calc in order to avoid an infinite calc loop. I'd be concerned about such a loop for this case if there was an unstable edje object somewhere in the chain...

Whatever you guys think. The latter seems simpler and less intensive. Let me know :)

I think probably if should be if (calculating) smart_calc(main_layout) else need_recalculate_set(main_layout) just to avoid potential infinite loops which might not occur with the default theme.

I think that's ok @stephenmhouston

Very odd you commented as I just updated.

Ok.