Opened 9 years ago

Closed 9 years ago

#5803 closed bug (fixed)

AudioFormatConverter ReadInt() is broken

Reported by: mmadia Owned by: stippi
Priority: normal Milestone: R1
Component: Applications/MediaPlayer Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

From Marcus on IRC: can somebody here have a look at AudioFormatConverter ReadInt() function? it's broken and uses short

short b = *(int*)buffer; 
if (b == 0x80000000)

mediaplayer/media_node_framework/audio/AudioFormatConverter.cpp. i was just browsing latest changes

Change History (3)

comment:1 by stippi, 9 years ago

Status: newin-progress

comment:2 by korli, 9 years ago

A fix is proposed in hrev36472.

comment:3 by stippi, 9 years ago

Resolution: fixed
Status: in-progressclosed

Yep, seems fine, thanks!

Note: See TracTickets for help on using tickets.