summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorJose Antonio Ortega Ruiz <jao@gnu.org>2004-06-06 19:48:26 +0000
committerJose Antonio Ortega Ruiz <jao@gnu.org>2004-06-06 19:48:26 +0000
commitfd33b7ceae50064f969e8a644fa05e5dfec20d87 (patch)
tree8c07a02190e6cf3c2d29965bd2177cf6488eaecc
parentff2ed93ec8860b5eea6b524d611f97149c6e5859 (diff)
downloadmdk-fd33b7ceae50064f969e8a644fa05e5dfec20d87.tar.gz
mdk-fd33b7ceae50064f969e8a644fa05e5dfec20d87.tar.bz2
bug fix: read only work in debug mode; more robust error reporting.
-rw-r--r--mixlib/mix_io.c81
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++)