<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Done: FS-5626 (<a href="http://jira.freeswitch.org/browse/FS-5626" style="font-family:arial">http://jira.freeswitch.org/browse/FS-5626</a>)</div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Markus</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">
<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/7/22 Ken Rice <span dir="ltr"><<a href="mailto:krice@freeswitch.org" target="_blank">krice@freeswitch.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
<font face="Monaco, Courier New"><span style="font-size:11pt">You should open a jira (<a href="http://jira.freeswitch.org" target="_blank">http://jira.freeswitch.org</a>) with this<div><div class="h5"><br>
<br>
<br>
On 7/22/13 2:10 AM, "Markus von Arx" <<a href="http://mkvonarx@gmail.com" target="_blank">mkvonarx@gmail.com</a>> wrote:<br>
<br>
</div></div></span></font><blockquote><span style="font-size:11pt"><div><div class="h5"><font face="Arial">Hi<br>
<br>
I tried to update to FreeSwitch 1.2.11 but get the following compile error (I'm compiling on Windows with Visual Studio 2012 Update 3):<br>
d:\freeswitch\build\freeswitch-1.2.11\src\switch_ivr_async.c(1207): error C2220: warning treated as error - no 'object' file generated [D:\FreeSwitch\Build\freeswitch-1.2.11\w32\Library\FreeSwitchCore.2012.vcxproj]<br>
d:\freeswitch\build\freeswitch-1.2.11\src\switch_ivr_async.c(1207): warning C4701: potentially uninitialized local variable 'len' used [D:\FreeSwitch\Build\freeswitch-1.2.11\w32\Library\FreeSwitchCore.2012.vcxproj]<br>
<br>
Checking the source code of switch_ivr_async.c I think the compiler is correct. In the source code, the lines 1213-1214 that contain the initialization of 'len' should probably be moved to line 1204.<br>
<br>
</font><font face="Monaco, Courier New">Also, the latest version of the file on github (<a href="https://github.com/FreeSWITCH/FreeSWITCH/blob/master/src/switch_ivr_async.c" target="_blank">https://github.com/FreeSWITCH/FreeSWITCH/blob/master/src/switch_ivr_async.c</a>) looks exactly like this (fixed with a commit from 3 July 2013: <a href="https://github.com/FreeSWITCH/FreeSWITCH/commit/02046950bebf84b6cf56f27321c3427718037c6b" target="_blank">https://github.com/FreeSWITCH/FreeSWITCH/commit/02046950bebf84b6cf56f27321c3427718037c6b</a>).<br>
<br>
</font><font face="Arial">=> What to do now with the 1.2.11 version? Will there be a 1.2.12 that fixes this compiler error?<br>
<br>
Here are the offending lines of code:<br>
1136: static switch_bool_t record_callback(switch_media_bug_t *bug, void *user_data, switch_abc_type_t type)<br>
...<br>
1143: switch_size_t len;<br>
...<br>
1199: case SWITCH_ABC_TYPE_TAP_NATIVE_WRITE:<br>
1200: {<br>
1201: switch_time_t now = switch_micro_time_now();<br>
1202: switch_time_t diff;<br>
1203: rh->wready = 1;<br>
1204: <br>
1205: if (!rh->rready) {<br>
1206: unsigned char fill_data[SWITCH_RECOMMENDED_BUFFER_SIZE] = {0};<br>
1207: switch_size_t fill_len = len;<br>
1208: switch_core_gen_encoded_silence(fill_data, &rh->read_impl, len);<br>
1209: switch_core_file_write(&rh->in_fh, fill_data, &fill_len);<br>
1210: }<br>
1211:<br>
1212:<br>
1213: nframe = switch_core_media_bug_get_native_write_frame(bug);<br>
1214: len = nframe->datalen;<br>
...<br>
<br>
Regards, Markus<br>
<br>
</font></div></div><font face="Monaco, Courier New"><br>
<hr align="CENTER" size="3" width="95%"></font></span><font><font face="Consolas, Courier New, Courier"><span style="font-size:10pt">_________________________________________________________________________<br>
Professional FreeSWITCH Consulting Services:<br>
<a href="http://consulting@freeswitch.org" target="_blank">consulting@freeswitch.org</a><br>
<a href="http://www.freeswitchsolutions.com" target="_blank">http://www.freeswitchsolutions.com</a><br>
<br>
FreeSWITCH-powered IP PBX: The CudaTel Communication Server<br>
<a href="http://www.cudatel.com" target="_blank">http://www.cudatel.com</a><br>
<br>
Official FreeSWITCH Sites<br>
<a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><br>
<a href="http://wiki.freeswitch.org" target="_blank">http://wiki.freeswitch.org</a><br>
<a href="http://www.cluecon.com" target="_blank">http://www.cluecon.com</a><br>
<br>
FreeSWITCH-users mailing list<br>
<a href="http://FreeSWITCH-users@lists.freeswitch.org" target="_blank">FreeSWITCH-users@lists.freeswitch.org</a><br>
<a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-users" target="_blank">http://lists.freeswitch.org/mailman/listinfo/freeswitch-users</a><br>
UNSUBSCRIBE:<a href="http://lists.freeswitch.org/mailman/options/freeswitch-users" target="_blank">http://lists.freeswitch.org/mailman/options/freeswitch-users</a><br>
<a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><span class="HOEnZb"><font color="#888888"><br>
</font></span></span></font></font></blockquote><span class="HOEnZb"><font color="#888888"><font><font face="Consolas, Courier New, Courier"><span style="font-size:10pt"><br>
</span></font></font><font face="Monaco, Courier New"><span style="font-size:11pt">-- <br>
Ken<br>
<font color="#0000FF"><u><a href="http://www.FreeSWITCH.org" target="_blank">http://www.FreeSWITCH.org</a><br>
<a href="http://www.ClueCon.com" target="_blank">http://www.ClueCon.com</a><br>
<a href="http://www.OSTAG.org" target="_blank">http://www.OSTAG.org</a><br>
</u></font><a href="http://irc.freenode.net" target="_blank">irc.freenode.net</a> #freeswitch<br>
</span></font>
</font></span></div>
<br>_________________________________________________________________________<br>
Professional FreeSWITCH Consulting Services:<br>
<a href="mailto:consulting@freeswitch.org">consulting@freeswitch.org</a><br>
<a href="http://www.freeswitchsolutions.com" target="_blank">http://www.freeswitchsolutions.com</a><br>
<br>
FreeSWITCH-powered IP PBX: The CudaTel Communication Server<br>
<a href="http://www.cudatel.com" target="_blank">http://www.cudatel.com</a><br>
<br>
Official FreeSWITCH Sites<br>
<a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><br>
<a href="http://wiki.freeswitch.org" target="_blank">http://wiki.freeswitch.org</a><br>
<a href="http://www.cluecon.com" target="_blank">http://www.cluecon.com</a><br>
<br>
FreeSWITCH-users mailing list<br>
<a href="mailto:FreeSWITCH-users@lists.freeswitch.org">FreeSWITCH-users@lists.freeswitch.org</a><br>
<a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-users" target="_blank">http://lists.freeswitch.org/mailman/listinfo/freeswitch-users</a><br>
UNSUBSCRIBE:<a href="http://lists.freeswitch.org/mailman/options/freeswitch-users" target="_blank">http://lists.freeswitch.org/mailman/options/freeswitch-users</a><br>
<a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><br>
<br></blockquote></div><br></div></div>