append_transient never returns 1

[dkadish]

voc/tract.w

Line 413 in 20a17d6

return 0;

Documentation states that append_transient should return 1 on success, but the function as written will never return 1. I think the last statement (line 413) should return 1. Unless I've missed something?

[PaulBatchelor]

Yup, the documentation is correct. The code is wrong. It would seem that the return code is never checked anywhere in the code.

[dkadish]

Ahh, got it. I'm working on a Python port of this (I'll post a link once I have something rough working), so I'm going over the code pretty closely. I'll let you know if I see anything else!

[PaulBatchelor]

Wow! Good luck. This code is quite a mess, as I was mostly trying to translate the core of Pink Trombone JS to ANSI C. So sorry about that.
It could use a rewrite or two, so I'm happy to see someone trying to tackle a rewrite in another language. Personally, I've been meaning to try to make a version of this (or portions of it) in FAUST. Someday I'll hopefully get to it...

If you make a PR with this fix, I'll be happy to merge it.