search

mntmn / interim

The Interim Operating System
1.24k stars 56 forks source link

Bail out on too long symbols #16

Closed wasamasa closed 5 years ago

wasamasa commented 7 years ago

I'm not too happy with allocating nil here, but it beats returning NULL (and getting a bogus cell warning).

Old behavior:

(def ffffffffffffffffffuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuu 42)
(gc) ;=> segfault

New behavior:

(def ffffffffffffffffffuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuu 42)
[insert_symbol] max symbol size exceeded by 6
nil