Opened 15 years ago
Closed 14 years ago
#1372 closed defect (fixed)
missing break in reorder_ch
| Reported by: | Owned by: | ||
|---|---|---|---|
| Priority: | normal | Component: | af |
| Version: | unspecified | Severity: | normal |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Reproduced by developer: | no | Analyzed by developer: | no |
Description
break for case 3 is missing in code in multiple lines in file libaf/reorder_ch.c
Attachments (1)
Change History (5)
by , 15 years ago
| Attachment: | mplayer-3bytes.diff added |
|---|
comment:1 by , 15 years ago
comment:2 by , 15 years ago
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
Quickly skimming it doesn't seem like your patch is solving any bug, does it?
Please explain. Just so you know; breaks are not actually mandatory.
comment:3 by , 15 years ago
| Resolution: | invalid |
|---|---|
| Status: | closed → reopened |
without patch stderr is flooded with following line if samplesize==3:
mp_msg(MSGT_GLOBAL, MSGL_WARN,
"[reorder_ch] Unsupported sample size: %d, please "
"report this error on the MPlayer mailing list.\n",samplesize);
Note:
See TracTickets
for help on using tickets.
patch