The initial motivation for this was the possibility of multiple arenas
per file, which obviously precludes having a single static arena.

But happily, in making the change I also fixed two other problems:

1. The tests no longer need to be run in a single thread.

2. The "mystery" of missing odd-numbered token ids was fixed by using
   mem::replace instead of Vec::insert inside TokenArena::set.