[PATCH:libXaw3d] Just use C89 size_t instead of rolling our own Size_t & Off_t
Hans de Goede
hdegoede at redhat.com
Thu Nov 6 06:03:21 PST 2014
Hi,
On 11/06/2014 03:26 AM, Alan Coopersmith wrote:
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
Looks good:
Reviewed-by: Hans de Goede <hdegoede at redhat.com>
Regards,
Hans
> ---
> src/AsciiSrc.c | 15 ++++-----------
> src/MultiSrc.c | 16 ++++------------
> 2 files changed, 8 insertions(+), 23 deletions(-)
>
> diff --git a/src/AsciiSrc.c b/src/AsciiSrc.c
> index 2f59441..ff7b9df 100644
> --- a/src/AsciiSrc.c
> +++ b/src/AsciiSrc.c
> @@ -115,13 +115,6 @@ static String StorePiecesInString(AsciiSrcObject);
> static Boolean SetValues(Widget, Widget, Widget, ArgList, Cardinal *);
> static Boolean WriteToFile(_Xconst _XtString, _Xconst _XtString);
>
> -#ifdef X_NOT_POSIX
> -#define Off_t long
> -#define Size_t unsigned int
> -#else
> -#define Off_t off_t
> -#define Size_t size_t
> -#endif
>
> #define superclass (&textSrcClassRec)
> AsciiSrcClassRec asciiSrcClassRec = {
> @@ -1039,7 +1032,7 @@ InitStringOrFile(AsciiSrcObject src, Boolean newString)
>
> if (!src->ascii_src.is_tempfile) {
> if ((file = fopen(src->ascii_src.string, open_mode)) != 0) {
> - (void) fseek(file, (Off_t)0, 2);
> + (void) fseek(file, 0, 2);
> src->ascii_src.length = (XawTextPosition) ftell(file);
> return file;
> } else {
> @@ -1069,9 +1062,9 @@ LoadPieces(AsciiSrcObject src, FILE * file, char * string)
> local_str = XtMalloc((unsigned) (src->ascii_src.length + 1)
> * sizeof(unsigned char));
> if (src->ascii_src.length != 0) {
> - fseek(file, (Off_t)0, 0);
> - src->ascii_src.length = fread(local_str, (Size_t)sizeof(unsigned char),
> - (Size_t)src->ascii_src.length, file);
> + fseek(file, 0, 0);
> + src->ascii_src.length = fread(local_str, sizeof(unsigned char),
> + (size_t)src->ascii_src.length, file);
> if (src->ascii_src.length <= 0)
> XtErrorMsg("readError", "asciiSourceCreate", "XawError",
> "fread returned error.", NULL, NULL);
> diff --git a/src/MultiSrc.c b/src/MultiSrc.c
> index c7cc2ac..3ea875b 100644
> --- a/src/MultiSrc.c
> +++ b/src/MultiSrc.c
> @@ -139,14 +139,6 @@ static void (MyWStrncpy)();
>
> extern char *tmpnam(String);
>
> -#ifdef X_NOT_POSIX
> -#define Off_t long
> -#define Size_t unsigned int
> -#else
> -#define Off_t off_t
> -#define Size_t size_t
> -#endif
> -
> extern wchar_t* _XawTextMBToWC(Display *, char *, int *);
> extern char *_XawTextWCToMB(Display *, wchar_t *, int *);
>
> @@ -1126,7 +1118,7 @@ InitStringOrFile(MultiSrcObject src, Boolean newString)
>
> if (!src->multi_src.is_tempfile) {
> if ((file = fopen(src->multi_src.string, open_mode)) != 0) {
> - (void) fseek(file, (Off_t)0, 2);
> + (void) fseek(file, 0, 2);
> src->multi_src.length = ftell (file);
> return file;
> } else {
> @@ -1190,10 +1182,10 @@ LoadPieces(MultiSrcObject src, FILE *file, char *string)
> if (src->multi_src.length != 0) {
> temp_mb_holder =
> XtMalloc((unsigned)(src->multi_src.length + 1) * sizeof(unsigned char));
> - fseek(file, (Off_t)0, 0);
> + fseek(file, 0, 0);
> src->multi_src.length = fread (temp_mb_holder,
> - (Size_t)sizeof(unsigned char),
> - (Size_t)src->multi_src.length, file);
> + sizeof(unsigned char),
> + (size_t)src->multi_src.length, file);
> if (src->multi_src.length <= 0)
> XtAppErrorMsg( XtWidgetToApplicationContext ((Widget) src),
> "readError", "multiSource", "XawError",
>
More information about the xorg-devel
mailing list