File gcc48.diff of Package spirv-tools

Overview Repositories Revisions Requests Users Attributes Meta

File gcc48.diff of Package spirv-tools (Revision 309d0516ae908cd1f2718347d7ca6e4c)

Currently displaying revision 309d0516ae908cd1f2718347d7ca6e4c , Show latest

From: Jan Engelhardt <jengelh@inai.de>
Date: 2017-02-02 19:42:44.679588946 +0100

build: workaround gcc 4.8 bug

tools/io.h:41:38:
error: ignoring return value of 'long int ftell(FILE*)',
declared with attribute warn_unused_result [-Werror=unused-result,-Werror]
       if (sizeof(T) != 1 && (ftell(fp) % sizeof(T))) {

Workaround gcc 4.8's wrongful idea that ftell's return value
were ignored.
---
 tools/io.h |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Index: spirv-tools-2018.4.g0/tools/io.h
===================================================================
--- spirv-tools-2018.4.g0.orig/tools/io.h
+++ spirv-tools-2018.4.g0/tools/io.h
@@ -38,7 +38,8 @@ bool ReadFile(const char* filename, cons
         return false;
       }
     } else {
-      if (sizeof(T) != 1 && (ftell(fp) % sizeof(T))) {
+      long ftx = ftell(fp);
+      if (sizeof(T) != 1 && ftx % sizeof(T) != 0) {
         fprintf(
             stderr,
             "error: file size should be a multiple of %zd; file '%s' corrupt\n",

Places

File gcc48.diff of Package spirv-tools (Revision 309d0516ae908cd1f2718347d7ca6e4c)

Places