<div dir="ltr"><br><div class="gmail_quote">2010/8/2 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net">remi@remlab.net</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im"><br>
On Mon,  2 Aug 2010 13:21:40 +0300, David Kaplan <<a href="mailto:david@2of1.org">david@2of1.org</a>> wrote:<br>
> +#define SATELLITE_TEXT N_("Satellite scanning config")<br>
> +#define SATELLITE_LONGTEXT N_("filename of config file in<br>
> share/dvb/dvb-s")<br>
<br>
</div>Beats me why you need a configuration item for this. Beats even harder how<br>
Joe Average User would know (to set) the right value.<br><div class="im"></div></blockquote><div><br>Well there needs to be some way of selecting the config file for the relevant satellite.<br>
<br>
I agree that average Joe wouldn't know what to put here - but then he 
wouldn't know what to put for most of the (currently) confusing options.<br><br>As part of the interface improvements I'm planning, I hope to provide a drop down list in a proper 'Scan' dialog.<br>But for now I think I need this option. Unless there's some other way I should be handling this?<br>
 </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
> -#define DVB_SCAN_MAX_SIGNAL_TIME (300*1000)<br>
> +#define DVB_SCAN_MAX_SIGNAL_TIME (1000*1000)<br>
>  #define DVB_SCAN_MAX_LOCK_TIME (5000*1000)<br>
> -#define DVB_SCAN_MAX_PROBE_TIME (30000*1000)<br>
> +#define DVB_SCAN_MAX_PROBE_TIME (45000*1000)<br>
<br>
</div>Undocumented and seemingly unrelated change.<br></blockquote><div><br>Will split out into a separate patch.<br> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div class="im"><br>
> +    if( p_scan->parameter.type == SCAN_DVB_S )<br>
> +        /* set satellite config file path */<br>
> +        strncpy( p_scan->parameter.sat_info.psz_name,<br>
> +                 var_GetString( p_access, "dvb-satellite" ),<br>
> +                 sizeof(p_scan->parameter.sat_info.psz_name) );<br>
<br>
</div>Memory leak.<br><div class="im"></div></blockquote><div><br>Where's it leaking? .psz_name is allocated on the heap.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">
<br>
> +#define GET_OPTION_STRING( option )<br>
<br>
>   \<br>
> +    if ( !strncmp( psz_parser, option "=", strlen( option "=" ) ) )<br>
<br>
>   \<br>
> +    {<br>
<br>
>   \<br>
> +        psz_parser += strlen( option "=" );<br>
<br>
>   \<br>
> +        val.psz_string = psz_parser;<br>
<br>
>   \<br>
> +        char *tok = strtok(val.psz_string, ":");<br>
<br>
<br>
</div>IIRC not thread-safe. Consider using strtok_r or something else.<br></blockquote><div><br>Indeed. WIll fix.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<br>
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net" target="_blank">http://www.remlab.net</a><br>
<a href="http://fi.linkedin.com/in/remidenis" target="_blank">http://fi.linkedin.com/in/remidenis</a><br>
<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr"><font size="1"><a style="font-family: courier new,monospace;" href="http://blog.2of1.org" target="_blank">http://blog.2of1.org</a></font><br></div><br>
</div>