<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Are you saying this has already been fixed, we just need to tag another release first.<div><br></div><div>Mike</div><div><br><div><div>On Jul 22, 2013, at 3:10 AM, Markus von Arx <<a href="mailto:mkvonarx@gmail.com">mkvonarx@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Hi</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">
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):</div><div class="gmail_default"><div class="gmail_default"><font face="arial, helvetica, sans-serif">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]</font></div>
<div class="gmail_default"><font face="arial, helvetica, sans-serif">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]</font></div>
<div style="font-family:arial,helvetica,sans-serif"><br></div><div style="font-family:arial,helvetica,sans-serif">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.</div>
<div style="font-family:arial,helvetica,sans-serif"><br></div>Also, the latest version of the file on github (<a href="https://github.com/FreeSWITCH/FreeSWITCH/blob/master/src/switch_ivr_async.c">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">https://github.com/FreeSWITCH/FreeSWITCH/commit/02046950bebf84b6cf56f27321c3427718037c6b</a>).</div>
<div class="gmail_default"><br><div style="font-family:arial,helvetica,sans-serif">=> What to do now with the 1.2.11 version? Will there be a 1.2.12 that fixes this compiler error?</div><div style="font-family:arial,helvetica,sans-serif">
<br></div><div style="font-family:arial,helvetica,sans-serif">Here are the offending lines of code:</div><div><div><font face="arial, helvetica, sans-serif">1136: static switch_bool_t record_callback(switch_media_bug_t *bug, void *user_data, switch_abc_type_t type)</font></div>
<div><font face="arial, helvetica, sans-serif">...</font></div><div><font face="arial, helvetica, sans-serif">1143: <span class="" style="white-space:pre"> </span>switch_size_t len;</font></div><div><font face="arial, helvetica, sans-serif">...</font></div>
<div><font face="arial, helvetica, sans-serif">1199:<span class="" style="white-space:pre"> </span>case SWITCH_ABC_TYPE_TAP_NATIVE_WRITE:</font></div><div><font face="arial, helvetica, sans-serif">1200:<span class="" style="white-space:pre"> </span>{</font></div>
<div><font face="arial, helvetica, sans-serif">1201:<span class="" style="white-space:pre"> </span>switch_time_t now = switch_micro_time_now();</font></div><div><font face="arial, helvetica, sans-serif">1202:<span class="" style="white-space:pre"> </span>switch_time_t diff;</font></div>
<div><font face="arial, helvetica, sans-serif">1203:<span class="" style="white-space:pre"> </span>rh->wready = 1;</font></div><div><font face="arial, helvetica, sans-serif">1204:<span class="" style="white-space:pre"> </span></font></div>
<div><font face="arial, helvetica, sans-serif">1205:<span class="" style="white-space:pre"> </span>if (!rh->rready) {</font></div><div><font face="arial, helvetica, sans-serif">1206:<span class="" style="white-space:pre"> </span>unsigned char fill_data[SWITCH_RECOMMENDED_BUFFER_SIZE] = {0};</font></div>
<div><font face="arial, helvetica, sans-serif">1207:<span class="" style="white-space:pre"> </span>switch_size_t fill_len = len;</font></div><div><font face="arial, helvetica, sans-serif">1208:<span class="" style="white-space:pre"> </span>switch_core_gen_encoded_silence(fill_data, &rh->read_impl, len);</font></div>
<div><font face="arial, helvetica, sans-serif">1209:<span class="" style="white-space:pre"> </span>switch_core_file_write(&rh->in_fh, fill_data, &fill_len);</font></div><div><font face="arial, helvetica, sans-serif">1210:<span class="" style="white-space:pre"> </span>}</font></div>
<div><font face="arial, helvetica, sans-serif">1211:</font></div><div><font face="arial, helvetica, sans-serif">1212:</font></div><div><font face="arial, helvetica, sans-serif">1213:<span class="" style="white-space:pre"> </span>nframe = switch_core_media_bug_get_native_write_frame(bug);</font></div>
<div><font face="arial, helvetica, sans-serif">1214:<span class="" style="white-space:pre"> </span>len = nframe->datalen;</font></div><div><font face="arial, helvetica, sans-serif">...</font></div></div><div><font face="arial, helvetica, sans-serif"><br>
</font></div><div><font face="arial, helvetica, sans-serif">Regards, Markus</font></div><div><font face="arial, helvetica, sans-serif"><br></font></div></div></div>
_________________________________________________________________________<br>Professional FreeSWITCH Consulting Services:<br><a href="mailto:consulting@freeswitch.org">consulting@freeswitch.org</a><br>http://www.freeswitchsolutions.com<br><br>FreeSWITCH-powered IP PBX: The CudaTel Communication Server<br>http://www.cudatel.com<br><br>Official FreeSWITCH Sites<br>http://www.freeswitch.org<br>http://wiki.freeswitch.org<br>http://www.cluecon.com<br><br>FreeSWITCH-users mailing list<br>FreeSWITCH-users@lists.freeswitch.org<br>http://lists.freeswitch.org/mailman/listinfo/freeswitch-users<br>UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-users<br>http://www.freeswitch.org<br></blockquote></div><br></div></body></html>