FNFT icon indicating copy to clipboard operation
FNFT copied to clipboard

eps_t unused in private/fnft__nsep_testcases.c

Open marbre opened this issue 6 years ago • 2 comments

Taking a deeper look into the code, I noticed that eps_t is initialized as https://github.com/FastNFT/FNFT/blob/bcd8767c99088d13c34f7b152ddea5447ee141c7/src/private/fnft__nsep_testcases.c#L187

However, eps_t seems not to be used afterwards.

marbre avatar Oct 09 '19 22:10 marbre

Seems like you are right. Will remove this in the next smaller release. Thanks!

wahls avatar Oct 10 '19 10:10 wahls

I just wasn't 100% certain, but in the case it is unused, I might send a PR and will refactor the eps_t usage in the specific file a little bit more than just deleting the line.

marbre avatar Oct 14 '19 08:10 marbre