diff options
author | Jose Antonio Ortega Ruiz <jao@gnu.org> | 2004-06-06 19:48:26 +0000 |
---|---|---|
committer | Jose Antonio Ortega Ruiz <jao@gnu.org> | 2004-06-06 19:48:26 +0000 |
commit | fd33b7ceae50064f969e8a644fa05e5dfec20d87 (patch) | |
tree | 8c07a02190e6cf3c2d29965bd2177cf6488eaecc /mixlib | |
parent | ff2ed93ec8860b5eea6b524d611f97149c6e5859 (diff) | |
download | mdk-fd33b7ceae50064f969e8a644fa05e5dfec20d87.tar.gz mdk-fd33b7ceae50064f969e8a644fa05e5dfec20d87.tar.bz2 |
bug fix: read only work in debug mode; more robust error reporting.
Diffstat (limited to 'mixlib')
-rw-r--r-- | mixlib/mix_io.c | 81 |
1 files changed, 30 insertions, 51 deletions
diff --git a/mixlib/mix_io.c b/mixlib/mix_io.c index 4a19abb..84b9ee0 100644 --- a/mixlib/mix_io.c +++ b/mixlib/mix_io.c @@ -1,22 +1,22 @@ /* -*-c-*- --------------- mix_io.c : * Implementation of the functions declared in mix_io.h * ------------------------------------------------------------------ - * Copyright (C) 2000 Free Software Foundation, Inc. - * + * Copyright (C) 2000, 2004 Free Software Foundation, Inc. + * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by * the Free Software Foundation; either version 2 of the License, or * (at your option) any later version. - * + * * This program is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. - * + * * You should have received a copy of the GNU General Public License * along with this program; if not, write to the Free Software * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. - * + * */ @@ -45,28 +45,28 @@ mix_io_delete (mix_iochannel_t *ch) FILE * mix_io_to_FILE (mix_iochannel_t *ioc) { - g_return_val_if_fail (ioc != NULL, FALSE); + if (ioc == NULL) return NULL; return ioc->file; } gboolean mix_io_eof (mix_iochannel_t *ioc) { - g_return_val_if_fail (ioc != NULL, FALSE); + if (ioc == NULL) return TRUE; return is_eof_ (ioc); } gboolean mix_io_is_ready (mix_iochannel_t *ioc) { - g_return_val_if_fail (ioc != NULL, FALSE); + if (ioc == NULL) return FALSE; return is_ready_ (ioc); } gboolean mix_io_write_byte (mix_iochannel_t *ioc, mix_byte_t b) { - g_return_val_if_fail (ioc != NULL, FALSE); + if (ioc == NULL) return FALSE; return write_data_ (ioc, &b, 1); } @@ -74,121 +74,101 @@ gboolean mix_io_write_byte_array (mix_iochannel_t *ioc, const mix_byte_t *b, size_t s) { - g_return_val_if_fail (ioc != NULL, FALSE); - g_return_val_if_fail (b != NULL, FALSE); + if (ioc == NULL || b == NULL) return FALSE; return write_data_ (ioc, b, s); } - mix_byte_t mix_io_read_byte (mix_iochannel_t *ioc) { mix_byte_t result = MIX_BYTE_ZERO; - g_return_val_if_fail (ioc != NULL, result); - g_return_val_if_fail (read_data_ (ioc, &result, 1), MIX_BYTE_ZERO); + if (ioc != NULL) read_data_ (ioc, &result, 1); return result; } - + gboolean mix_io_read_byte_array (mix_iochannel_t *ioc, mix_byte_t *b, size_t s) { - g_return_val_if_fail (ioc != NULL, FALSE); - g_return_val_if_fail (b != NULL, FALSE); - return read_data_ (ioc, b, s); + return (ioc != NULL) && (b != NULL) && read_data_ (ioc, b, s); } gboolean mix_io_write_word (mix_iochannel_t *ioc, mix_word_t w) { - g_return_val_if_fail (ioc != NULL, FALSE); - return write_data_ (ioc, &w, 1); + return (ioc != NULL) && write_data_ (ioc, &w, 1); } gboolean mix_io_write_word_array (mix_iochannel_t *ioc, const mix_word_t *w, size_t s) { - g_return_val_if_fail (ioc != NULL, FALSE); - g_return_val_if_fail (w != NULL, FALSE); - return write_data_ (ioc, w, s); + return (ioc != NULL) && (w != NULL) && write_data_ (ioc, w, s); } mix_word_t mix_io_read_word (mix_iochannel_t *ioc) { mix_word_t result = MIX_WORD_ZERO; - g_return_val_if_fail (ioc != NULL, result); - g_return_val_if_fail (read_data_ (ioc, &result, 1), MIX_WORD_ZERO); + if (ioc != NULL) read_data_ (ioc, &result, 1); return result; } gboolean mix_io_read_word_array (mix_iochannel_t *ioc, mix_word_t *w, size_t s) { - g_return_val_if_fail (ioc != NULL, FALSE); - g_return_val_if_fail (w != NULL, FALSE); - return read_data_ (ioc, w, s); + return (ioc != NULL) && (w != NULL) && read_data_ (ioc, w, s); } gboolean mix_io_write_short (mix_iochannel_t *ioc, mix_short_t w) { - g_return_val_if_fail (ioc != NULL, FALSE); - return write_data_ (ioc, &w, 1); + return (ioc != NULL) && write_data_ (ioc, &w, 1); } gboolean mix_io_write_short_array (mix_iochannel_t *ioc, const mix_short_t *w, size_t s) { - g_return_val_if_fail (ioc != NULL, FALSE); - g_return_val_if_fail (w != NULL, FALSE); - return write_data_ (ioc, w, s); + return (ioc != NULL) && (w != NULL) && write_data_ (ioc, w, s); } mix_short_t mix_io_read_short (mix_iochannel_t *ioc) { mix_short_t result = MIX_SHORT_ZERO; - g_return_val_if_fail (ioc != NULL, result); - g_return_val_if_fail (read_data_ (ioc, &result, 1), MIX_SHORT_ZERO); + if (ioc != NULL) read_data_ (ioc, &result, 1); return result; } gboolean mix_io_read_short_array (mix_iochannel_t *ioc, mix_short_t *w, size_t s) { - g_return_val_if_fail (ioc != NULL, FALSE); - g_return_val_if_fail (w != NULL, FALSE); - return read_data_ (ioc, w, s); + return (ioc != NULL) && (w != NULL) && read_data_ (ioc, w, s); } gboolean mix_io_write_char (mix_iochannel_t *ioc, mix_char_t c) { guchar value = mix_char_to_ascii (c); - g_return_val_if_fail (ioc != NULL, FALSE); - return write_data_ (ioc, &value, 1); + return (ioc != NULL) && write_data_ (ioc, &value, 1); } mix_char_t mix_io_read_char (mix_iochannel_t *ioc) { - guchar value; - g_return_val_if_fail (ioc != NULL, MIX_CHAR_MAX); - g_return_val_if_fail (read_data_ (ioc, &value, 1), MIX_CHAR_MAX); + guchar value = MIX_CHAR_MAX; + if (ioc != NULL) read_data_ (ioc, &value, 1); return mix_ascii_to_char (value); } gboolean -mix_io_write_word_array_as_char (mix_iochannel_t *ioc, +mix_io_write_word_array_as_char (mix_iochannel_t *ioc, const mix_word_t *w, size_t s) { guint k, j; - - g_return_val_if_fail (ioc != NULL, FALSE); - g_return_val_if_fail (w != NULL, FALSE); - + + if ((ioc == NULL) || (w == NULL)) return FALSE; + for (k = 0; k < s; k++) for (j = 1; j < 6; j++) { @@ -204,9 +184,8 @@ mix_io_read_word_array_as_char (mix_iochannel_t *ioc, mix_word_t *w, size_t s) { guint k, j; - - g_return_val_if_fail (ioc != NULL, FALSE); - g_return_val_if_fail (w != NULL, FALSE); + + if ((ioc == NULL) || (w == NULL)) return FALSE; for (k = 0; k < s; k++) for (j = 1; j < 6; j++) |