Ampersand just needs to be removed from argument passed to video_s3_load_video() on line 27. I'll submit a patch.

CommentFileSizeAuthor
#1 pass_by_ref_bug-1083104-1.patch800 bytesrohnjeynolds

Comments

rohnjeynolds’s picture

rohnjeynolds commented
StatusFileSize
new pass_by_ref_bug-1083104-1.patch800 bytes

Patch attached

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented

Thanks for the patches, we are developing the core of the video module for D7 and yet to start S3 and Zencoder integration. You may know in D7 the stream wrappers comes in to play, so my plan is to create a stream wrapper for S3 and use directly it with D7.

I will push some code in couple of hours, which is the later we have.

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Status: Active » Closed (fixed)