[Kde-pim] KAlarm : Potentially dangerous compile warnings

[David Jarvie]

On Tue 17 June 2008 16:25:41 Jaroslaw Staniek wrote:
> [ 53%] Building CXX object kalarm/CMakeFiles/kalarm_bin.dir/alarmevent.obj
> C:\kde4\svn\branches\kdepim\enterprise4\kdepim\kalarm\alarmevent.cpp(4008)
> : warning C4804: '%' : unsafe use of type 'bool' in operation
> c:\kde4\svn\branches\kdepim\enterprise4\kdepim\kalarm\alarmevent.h(317) :
> warning C4717: 'KAEvent::set' : recursive on all control paths, function
> will cause runtime stack overflow
> [ 53%] Building CXX object kalarm/CMakeFiles/kalarm_bin.dir/editdlg.obj
> C:\kde4\svn\branches\kdepim\enterprise4\kdepim\kalarm\editdlg.cpp(988) :
> warning C4804: '/' : unsafe use of type 'bool'
> in operation

These are now fixed - thanks for pointing them out. It turns out that a few 
other (unused) methods in alarmevent.h also would have been recursive in the 
same way, but strangely only one was reported by the compiler.

-- 
David Jarvie.
KAlarm author and maintainer.
http://www.astrojar.org.uk/kalarm
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/