Re: [FFmpeg-devel] WinRT API support patch

2014-11-21 Thread Reimar Döffinger
rds,Jesse >> From: reimar.doeffin...@gmx.de >> Date: Thu, 20 Nov 2014 09:18:30 +0100 >> To: ffmpeg-devel@ffmpeg.org >> Subject: Re: [FFmpeg-devel] WinRT API support patch >> >> On 20 November 2014 08:34:54 CET, Jesse Jiang >> wrote: >>> Add WinR

Re: [FFmpeg-devel] WinRT API support patch

2014-11-21 Thread Reimar Döffinger
;) == sizeof(seed)) > return seed;return get_generic_seed();} >> From: jessejiang0...@outlook.com >> To: ffmpeg-devel@ffmpeg.org >> Date: Fri, 21 Nov 2014 03:51:12 + >> Subject: Re: [FFmpeg-devel] WinRT API support patch >> >> Hi All, >>

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Jesse Jiang
") == sizeof(seed))return seed; if (read_random(&seed, "/dev/random") == sizeof(seed))return seed; return get_generic_seed();} > From: jessejiang0...@outlook.com > To: ffmpeg-devel@ffmpeg.org > Date: Fri, 21 Nov 2014 03:51:12 + > Subject: Re

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Jesse Jiang
any way to instead of srand(time(0)); rand(); ? How about c++11 or Mersenne twister Algorithmic Best regards,Jesse Date: Fri, 21 Nov 2014 02:49:59 +0100 From: michae...@gmx.at To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] WinRT API support patch On Fri, Nov 21, 2014 at 01:43:29AM +

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Michael Niedermayer
On Fri, Nov 21, 2014 at 01:43:29AM +, Jesse Jiang wrote: > Do you mean that we can use rand() instead of CryptGenRandom in ffmpeg? rand() is completely wrong its not even doing the correct operation rand() is pseudo random the code requires a strong (and non pseudo) random value [...] -- Mi

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Jesse Jiang
Do you mean that we can use rand() instead of CryptGenRandom in ffmpeg? > Date: Thu, 20 Nov 2014 12:17:10 +0100 > From: j...@videolan.org > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] WinRT API support patch > > 1. libgcc is not big > 2. MSVC 2013 has correct

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Jean-Baptiste Kempf
a big library with our Apps. I know the mingw is a correct way, but how could it works with MSVC? Date: Thu, 20 Nov 2014 09:53:27 +0100 From: j...@videolan.org To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] WinRT API support patch On 20 Nov, Jesse Jiang wrote : Add WinRT API sup

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Jesse Jiang
an.org > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] WinRT API support patch > > On 20 Nov, Jesse Jiang wrote : > > Add WinRT API supports > > This patch is wrong on almost every level. > > rand() is wrong. >

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Jean-Baptiste Kempf
On 20 Nov, Jesse Jiang wrote : > Add WinRT API supports This patch is wrong on almost every level. rand() is wrong. modifying configure because your headers are broken is wrong too. doing an API rewrapper in FFmpeg is the wrong place, look at winstorecomp

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Jean-Baptiste Kempf
function > instead of it. If we use WinRT api to instead of these apis, it will cause > more bugs. > Best regards,Jesse > > From: reimar.doeffin...@gmx.de > > Date: Thu, 20 Nov 2014 09:18:30 +0100 > > To: ffmpeg-devel@ffmpeg.org > > Subject: Re: [FFmpeg-devel] WinRT API

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Jesse Jiang
.de > Date: Thu, 20 Nov 2014 09:18:30 +0100 > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] WinRT API support patch > > On 20 November 2014 08:34:54 CET, Jesse Jiang > wrote: > >Add WinRT API supports > > At least 2 fairly major is

Re: [FFmpeg-devel] WinRT API support patch

2014-11-20 Thread Reimar Döffinger
On 20 November 2014 08:34:54 CET, Jesse Jiang wrote: >Add WinRT API supports At least 2 fairly major issues: 1) using rand() basically never is correct. Use the appropriate CryptographicBuffer function (yes, that means you need some C++ code unfortunately). 2) CreateTh