Opened 8 years ago

Closed 8 years ago

#7341 closed bug (fixed)

[mediaconverter] doesn't report correct file details

Reported by: starsseed Owned by: stippi
Priority: normal Milestone: R1
Component: Applications/MediaConverter Version: R1/alpha2
Keywords: r40649 Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

trunk\src\apps\mediaconverter\MediaFileInfo.cpp line 73 :
the format string
"%Ld x %Ld, %Ld fps / %Ld frames"
is not correct :
for a 320*240 movie, the reported width is 1030792151360 ( 320 + 240*232 )

i.e. "%Ld" read 8 bytes on the stack instead of 4.

Attachments (1)

MediaConverter.patch (1.4 KB) - added by starsseed 8 years ago.
patch for 5 type specifier errors in 3 snprintf format string

Download all attachments as: .zip

Change History (5)

comment:1 Changed 8 years ago by starsseed

Keywords: r40649 added

comment:2 Changed 8 years ago by starsseed

NB :

fps is a double or a float (not an integer)

The same way :

in trunk\src\apps\mediaconverter\MediaConverterApp.cpp line 614 :

"Writing audio track: %Ld %% complete"

and trunk\src\apps\mediaconverter\MediaConverterApp.cpp line 556 :

"Writing video track: %Ld %% complete"

%LD should by replaced by %ld

Changed 8 years ago by starsseed

Attachment: MediaConverter.patch added

patch for 5 type specifier errors in 3 snprintf format string

comment:3 Changed 8 years ago by starsseed

Has a Patch: set

comment:4 Changed 8 years ago by stippi

Resolution: fixed
Status: newclosed

Thanks for the patch! Applied in hrev40876.

Note: See TracTickets for help on using tickets.