Subject: | toke_scan_str doesn't handle errors, returns garbage |
toke_scan_str returns bogus values such as -141784940 if parsing fails.
I think this is because S_scan_str returns NULL, which is not handled by
dd_toke_scan_str.
My preferred fix would be making toke_scan_str return undef in case of
failure, but even a simple 'if (!s) { return -1; }' in dd_toke_scan_str
would be better than the current behavior.