Does this test against -1 need a cast to offset_t (as other places do)