discuss-gnustep
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Testers wanted for MPDCon and Zipper


From: Fred Kiefer
Subject: Re: Testers wanted for MPDCon and Zipper
Date: Wed, 22 Feb 2012 18:07:13 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120208 Thunderbird/10.0.1

Looks like you fixed some more bugs since this mail. Most of he still remaining warnings should be caused by NSNotFound being of type NSUInteger and the code is using it as an int. Just change all these places to NSUInteger.

On 22.02.2012 15:51, Sebastian Reitenbach wrote:

On Wednesday, February 22, 2012 15:07 CET, Riccardo Mottola<address@hidden>  
wrote:

Hi,

Sebastian Reitenbach wrote:

On Wednesday, February 22, 2012 12:01 CET, Riccardo Mottola<address@hidden>   
wrote:

Hi,
I am not able to reproduce this behaviour here (OpenSuse 12.1, gcc 4.6,
64bit). I get plenty of horrible compiler warnings from Zipper
(current CVS), which really should be fixed before looking deeper into
any issue with that application.
Could you please describe in detail all the steps required to
reproduce the crash?

which warnings do you get? I don't get any on OpenBSD with gcc 4.2. On
NetBSD with 4.5 I get some harmless looking ones. I'll try to fix them
though.
With clang, I also get no warnings, with gcc-4.6 it looks like this:

I think I found some inconsistencies in some types. I blindly fixed

them, since I get no warnings neither before nor after. Do you mind to
check if I reduced them (or created new ones...)

Riccardo

  now it looks a bit different:

  gcc Archive.m -c \
       -MMD -MP -DGNUSTEP -DGNUSTEP_BASE_LIBRARY=1 -DGNU_GUI_LIBRARY=1 
-DGNU_RUNTIME=1 -DGNUSTEP_BASE_LIBRARY=1 -fno-strict-aliasing -fexceptions 
-fobjc-exceptions -D_NATIVE_OBJC_EXCEPTIONS -pthread -fPIC -Wall -DGSWARN 
-DGSDIAGNOSE -Wno-import -g -O2 -fgnu-runtime -Wall -Wno-import 
-fconstant-string-class=NSConstantString -fexec-charset=UTF-8 -I. 
-I/root/GNUstep/Library/Headers -I/opt/gnustep/include \
        -o obj/Zipper.obj/Archive.m.o
Archive.m: In function ‘-[Archive initWithPath:]’:
Archive.m:120:5: warning: overflow in implicit constant conversion [-Woverflow]
Archive.m: In function ‘-[Archive sortByPath]’:
Archive.m:171:21: warning: comparison between ‘NSComparisonResult’ and 
‘enum<anonymous>’ [-Wenum-compare]
Archive.m:177:17: warning: comparison between ‘SortByOptions’ and ‘enum 
_NSComparisonResult’ [-Wenum-compare]
Archive.m: In function ‘-[Archive sortBySize]’:
Archive.m:197:21: warning: comparison between ‘NSComparisonResult’ and 
‘enum<anonymous>’ [-Wenum-compare]
Archive.m:203:17: warning: comparison between ‘SortByOptions’ and ‘enum 
_NSComparisonResult’ [-Wenum-compare]
Archive.m: In function ‘-[Archive sortByFilename]’:
Archive.m:224:21: warning: comparison between ‘NSComparisonResult’ and 
‘enum<anonymous>’ [-Wenum-compare]
Archive.m:230:17: warning: comparison between ‘SortByOptions’ and ‘enum 
_NSComparisonResult’ [-Wenum-compare]
Archive.m: In function ‘-[Archive sortByDate]’:
Archive.m:250:21: warning: comparison between ‘NSComparisonResult’ and 
‘enum<anonymous>’ [-Wenum-compare]
Archive.m:256:17: warning: comparison between ‘SortByOptions’ and ‘enum 
_NSComparisonResult’ [-Wenum-compare]
Archive.m: In function ‘-[Archive sortByRatio]’:
Archive.m:276:21: warning: comparison between ‘NSComparisonResult’ and 
‘enum<anonymous>’ [-Wenum-compare]
Archive.m:282:17: warning: comparison between ‘SortByOptions’ and ‘enum 
_NSComparisonResult’ [-Wenum-compare]
Archive.m: In function ‘-[Archive expandFiles:withPathInfo:toPath:]’:
Archive.m:307:2: warning: function with qualified void return type called 
[enabled by default]
Archive.m:309:2: warning: overflow in implicit constant conversion [-Woverflow]
Archive.m: In function ‘-[Archive listContents]’:
Archive.m:371:2: warning: function with qualified void return type called 
[enabled by default]
Archive.m: In function ‘+[Archive unarchiveExecutable]’:
Archive.m:394:2: warning: function with qualified void return type called 
[enabled by default]
Archive.m: In function ‘+[Archive archiveType]’:
Archive.m:405:2: warning: function with qualified void return type called 
[enabled by default]

gcc FileInfo.m -c \
       -MMD -MP -DGNUSTEP -DGNUSTEP_BASE_LIBRARY=1 -DGNU_GUI_LIBRARY=1 
-DGNU_RUNTIME=1 -DGNUSTEP_BASE_LIBRARY=1 -fno-strict-aliasing -fexceptions 
-fobjc-exceptions -D_NATIVE_OBJC_EXCEPTIONS -pthread -fPIC -Wall -DGSWARN 
-DGSDIAGNOSE -Wno-import -g -O2 -fgnu-runtime -Wall -Wno-import 
-fconstant-string-class=NSConstantString -fexec-charset=UTF-8 -I. 
-I/root/GNUstep/Library/Headers -I/opt/gnustep/include \
        -o obj/Zipper.obj/FileInfo.m.o
FileInfo.m: In function ‘-[FileInfo comparePathAscending:]’:
FileInfo.m:145:2: warning: overflow in implicit constant conversion [-Woverflow]
FileInfo.m:172:2: warning: overflow in implicit constant conversion [-Woverflow]
FileInfo.m: In function ‘-[FileInfo compareSizeAscending:]’:
FileInfo.m:190:2: warning: overflow in implicit constant conversion [-Woverflow]
FileInfo.m: In function ‘-[FileInfo compareFilenameAscending:]’:
FileInfo.m:209:2: warning: overflow in implicit constant conversion [-Woverflow]
FileInfo.m: In function ‘-[FileInfo compareDateAscending:]’:
FileInfo.m:227:2: warning: overflow in implicit constant conversion [-Woverflow]
FileInfo.m: In function ‘-[FileInfo compareRatioAscending:]’:
FileInfo.m:245:2: warning: overflow in implicit constant conversion [-Woverflow]

gcc NSFileManager+Custom.m -c \
       -MMD -MP -DGNUSTEP -DGNUSTEP_BASE_LIBRARY=1 -DGNU_GUI_LIBRARY=1 
-DGNU_RUNTIME=1 -DGNUSTEP_BASE_LIBRARY=1 -fno-strict-aliasing -fexceptions 
-fobjc-exceptions -D_NATIVE_OBJC_EXCEPTIONS -pthread -fPIC -Wall -DGSWARN 
-DGSDIAGNOSE -Wno-import -g -O2 -fgnu-runtime -Wall -Wno-import 
-fconstant-string-class=NSConstantString -fexec-charset=UTF-8 -I. 
-I/root/GNUstep/Library/Headers -I/opt/gnustep/include \
        -o obj/Zipper.obj/NSFileManager+Custom.m.o
NSFileManager+Custom.m: In function ‘-[NSFileManager(Custom) 
locateExecutable:]’:
NSFileManager+Custom.m:26:8: warning: variable ‘isScanned’ set but not used 
[-Wunused-but-set-variable]

gcc TableViewDataSource.m -c \
       -MMD -MP -DGNUSTEP -DGNUSTEP_BASE_LIBRARY=1 -DGNU_GUI_LIBRARY=1 
-DGNU_RUNTIME=1 -DGNUSTEP_BASE_LIBRARY=1 -fno-strict-aliasing -fexceptions 
-fobjc-exceptions -D_NATIVE_OBJC_EXCEPTIONS -pthread -fPIC -Wall -DGSWARN 
-DGSDIAGNOSE -Wno-import -g -O2 -fgnu-runtime -Wall -Wno-import 
-fconstant-string-class=NSConstantString -fexec-charset=UTF-8 -I. 
-I/root/GNUstep/Library/Headers -I/opt/gnustep/include \
        -o obj/Zipper.obj/TableViewDataSource.m.o
TableViewDataSource.m: In function ‘-[TableViewDataSource 
tableView:willDisplayCell:forTableColumn:row:]’:
TableViewDataSource.m:27:9: warning: variable ‘size’ set but not used 
[-Wunused-but-set-variable]

ZipperDocument.m: In function ‘-[ZipperDocument openFile:withDefaultApp:]’:
ZipperDocument.m:431:12: warning: variable ‘extension’ set but not used 
[-Wunused-but-set-variable]





reply via email to

[Prev in Thread] Current Thread [Next in Thread]