Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-15 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 04:01:05PM +0200, Michael Niedermayer wrote: > MSVC does not allow passing file pointers between libs > This API can thus not work with MSVC and as it was very recently added > and its it was in no release its removial should not cause any problems > > Found-by: Hendrik Lep

Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-15 Thread Nicolas George
Le sextidi 26 messidor, an CCXXII, Michael Niedermayer a écrit : > an alternative suggestion or after this patch > would be to switch to passing a filename string instead of a int fd > and treat "" as stdin "-" for stdin is more standard. Otherwise, it would be completely consistent with the av_fi

Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 04:01:05PM +0200, Michael Niedermayer wrote: > MSVC does not allow passing file pointers between libs > This API can thus not work with MSVC and as it was very recently added > and its it was in no release its removial should not cause any problems > > Found-by: Hendrik Lep

Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 5:21 PM, Michael Niedermayer wrote: > but id like to remove thi file descriptor based API before the relase > so noone has to hit the same msvc issue I am OK with it, but others may want to comment, particularly its author. - Derek ___ ffmp

Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Michael Niedermayer
On Sun, Jul 13, 2014 at 04:54:00PM +0100, Derek Buitenhuis wrote: > On 7/13/2014 3:01 PM, Michael Niedermayer wrote: > > MSVC does not allow passing file pointers between libs > > This API can thus not work with MSVC and as it was very recently added > > and its it was in no release its removial sh

Re: [FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Derek Buitenhuis
On 7/13/2014 3:01 PM, Michael Niedermayer wrote: > MSVC does not allow passing file pointers between libs > This API can thus not work with MSVC and as it was very recently added > and its it was in no release its removial should not cause any problems > > Found-by: Hendrik Leppkes > Signed-off-b

[FFmpeg-devel] [PATCH] avutil & avdevice: remove av_bprint_fd_contents()

2014-07-13 Thread Michael Niedermayer
MSVC does not allow passing file pointers between libs This API can thus not work with MSVC and as it was very recently added and its it was in no release its removial should not cause any problems Found-by: Hendrik Leppkes Signed-off-by: Michael Niedermayer --- libavdevice/lavfi.c | 39 +