<div>Hello!</div><div><br>On Wed, Oct 26, 2016 at 8:31PM, Maxim Dounin wrote:</div><div><br>>On Wed, Oct 26, 2016 at 05:10:25PM +0800,  (hucc) wrote:<br>> <br>>> VOD (video on demand) module which support requesting time range<br>>> also need the special function, so i would like to see that nginx-devel<br>>> add a function in Core, that would be helpful. Possible patch below.<br>><br>>If you mean https://github.com/kaltura/nginx-vod-module, then I <br>>don't think it does.  Quick look suggests that it uses <br>>milliseconds to specify times instead, and uses ngx_atoi() to <br>>parse them.  It doesn't use strtod() at all, and the only place it <br>>uses ngx_atofp() now is to parse rate, and I don't think there are <br>>reasons to allow arbitrary numbers like in the mp4 case.<br>><br>>The main problem with mp4 was that there were existing Flash <br>>clients nginx has to be compatible with, and some of them used to <br>>send incorrect start time specifications.  It is highly unlikely <br>>that the same unique situation affects other modules.<br><br>I don`t know how to make a more specific description because of my</div><div>poor english.  What i mean about VOD is video-processing module in</div><div>CDN provider, this scenario usually be asked to provide parameter</div><div>parsing which support seconds and milliseconds.  I am developing a</div><div>series of moudles which are a bit like nginx-vod-module and new</div><div>function will be used based on my experience.  I just think the usage</div><div>of atofp() is very limited, maybe a function supporting optional mode</div><div>should be provided.  Anyway, the point is whether other modules</div><div>need new function, i pointed out the scene i familiar with and hope</div><div>this helps.</div><div><br></div><div><div style="font-family: "lucida Grande", Verdana;"><font face="lucida Grande, Verdana" style="line-height: 23.8px;">Regards,</font></div><div style="font-family: "lucida Grande", Verdana;"><font face="lucida Grande, Verdana" style="line-height: 23.8px;">-hucc</font></div></div>