Summary:	ASTERISK-14599: [patch] res_calendar.c does not compile in devmode
Reporter:	Michiel van Baak (mvanbaak)	Labels:
Date Opened:	2009-08-05 03:12:29	Date Closed:	2009-08-15 07:17:50
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Resources/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) 2009080400-rescalendarcompilefix.diff.txt ( 1) 2009081100-rescalendarcompilefix.diff.txt
Description:	When compiling res_calendar.c I get a bunch of warnings about different types. This patch fixes it for me. I'm not sure this is the correct fix though. So advice is welcome.
Comments:	By: Michiel van Baak (mvanbaak) 2009-08-11 03:02:11 Better way to fix it: cast all the time_t items to long where needed. This is more in line with the rest of the asterisk codebase. By: Digium Subversion (svnbot) 2009-08-15 06:36:46 Repository: asterisk Revision: 212343 U   trunk/res/res_calendar.c ------------------------------------------------------------------------ r212343 | mvanbaak | 2009-08-15 06:36:46 -0500 (Sat, 15 Aug 2009) | 10 lines cast time_t type variables to long where needed. This makes res_calendar.c compile on OpenBSD and the same cast is used in a lot of other places where time_t type vars are used. (closes issue ASTERISK-14599) Reported by: mvanbaak Patches:      2009081100-rescalendarcompilefix.diff.txt uploaded by mvanbaak (license 7) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=212343 By: Digium Subversion (svnbot) 2009-08-15 06:45:23 Repository: asterisk Revision: 212344 _U  branches/1.6.0/ ------------------------------------------------------------------------ r212344 | mvanbaak | 2009-08-15 06:38:12 -0500 (Sat, 15 Aug 2009) | 16 lines Blocked revisions 212343 via svnmerge ........  r212343 | mvanbaak | 2009-08-15 13:36:19 +0200 (Sat, 15 Aug 2009) | 10 lines    cast time_t type variables to long where needed.    This makes res_calendar.c compile on OpenBSD and the same  cast is used in a lot of other places where time_t type vars are used.    (closes issue ASTERISK-14599)  Reported by: mvanbaak  Patches:        2009081100-rescalendarcompilefix.diff.txt uploaded by mvanbaak (license 7) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=212344 By: Digium Subversion (svnbot) 2009-08-15 07:17:47 Repository: asterisk Revision: 212345 _U  branches/1.6.1/ ------------------------------------------------------------------------ r212345 | mvanbaak | 2009-08-15 07:17:46 -0500 (Sat, 15 Aug 2009) | 16 lines Blocked revisions 212343 via svnmerge ........  r212343 | mvanbaak | 2009-08-15 13:36:19 +0200 (Sat, 15 Aug 2009) | 10 lines    cast time_t type variables to long where needed.    This makes res_calendar.c compile on OpenBSD and the same  cast is used in a lot of other places where time_t type vars are used.    (closes issue ASTERISK-14599)  Reported by: mvanbaak  Patches:        2009081100-rescalendarcompilefix.diff.txt uploaded by mvanbaak (license 7) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=212345 By: Digium Subversion (svnbot) 2009-08-15 07:17:50 Repository: asterisk Revision: 212346 _U  branches/1.6.2/ ------------------------------------------------------------------------ r212346 | mvanbaak | 2009-08-15 07:17:50 -0500 (Sat, 15 Aug 2009) | 16 lines Blocked revisions 212343 via svnmerge ........  r212343 | mvanbaak | 2009-08-15 13:36:19 +0200 (Sat, 15 Aug 2009) | 10 lines    cast time_t type variables to long where needed.    This makes res_calendar.c compile on OpenBSD and the same  cast is used in a lot of other places where time_t type vars are used.    (closes issue ASTERISK-14599)  Reported by: mvanbaak  Patches:        2009081100-rescalendarcompilefix.diff.txt uploaded by mvanbaak (license 7) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=212346