From 2446dbc3b48e565dd677bd081440d6d522ed6908 Mon Sep 17 00:00:00 2001 From: "Reece H. Dunn" Date: Mon, 11 Jan 2016 21:26:39 +0000 Subject: [PATCH] Make LoadFrame return an error_ng_STATUS code. --- src/libespeak-ng/spect.c | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/src/libespeak-ng/spect.c b/src/libespeak-ng/spect.c index b9182d97..9aeed7ce 100644 --- a/src/libespeak-ng/spect.c +++ b/src/libespeak-ng/spect.c @@ -139,7 +139,7 @@ static void SpectFrameDestroy(SpectFrame *frame) free(frame); } -int LoadFrame(SpectFrame *frame, FILE *stream, int file_format_type) +static espeak_ng_STATUS LoadFrame(SpectFrame *frame, FILE *stream, int file_format_type) { short ix; short x; @@ -182,10 +182,8 @@ int LoadFrame(SpectFrame *frame, FILE *stream, int file_format_type) spect_data = malloc(sizeof(USHORT) * frame->nx); - if (spect_data == NULL) { - fprintf(stderr, "Failed to allocate memory\n"); - return 1; - } + if (spect_data == NULL) + return ENOMEM; frame->max_y = 0; for (ix = 0; ix < frame->nx; ix++) { @@ -195,7 +193,7 @@ int LoadFrame(SpectFrame *frame, FILE *stream, int file_format_type) } frame->spect = spect_data; - return 0; + return ENS_OK; } double GetFrameRms(SpectFrame *frame, int seq_amplitude) @@ -342,9 +340,10 @@ espeak_ng_STATUS LoadSpectSeq(SpectSeq *spect, const char *filename) if (!frame) return ENOMEM; - if (LoadFrame(frame, stream, spect->file_format) != 0) { + espeak_ng_STATUS status = LoadFrame(frame, stream, spect->file_format); + if (status != ENS_OK) { free(frame); - break; + return status; } spect->frames[spect->numframes++] = frame;