Opened 9 years ago

Closed 9 years ago

Last modified 6 years ago

#895 closed defect (fixed)

sunras: support 32bpp

Reported by: ami_stuff Owned by:
Priority: normal Component: avcodec
Version: git-master Keywords: sunrast
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

PIX_FMT_RGBA works correctly on big-endian, PIX_FMT_BGRA on little-endian

C:\>ffmpeg -i 32.ras
ffmpeg version N-36492-g7f83db3, Copyright (c) 2000-2012 the FFmpeg developers
  built on Jan  5 2012 01:57:12 with gcc 4.6.2
  configuration: --enable-gpl --enable-version3 --disable-w32threads --enable-ru
ntime-cpudetect --enable-avisynth --enable-bzlib --enable-frei0r --enable-libope
ncore-amrnb --enable-libopencore-amrwb --enable-libfreetype --enable-libgsm --en
able-libmp3lame --enable-libopenjpeg --enable-librtmp --enable-libschroedinger -
-enable-libspeex --enable-libtheora --enable-libvo-aacenc --enable-libvo-amrwben
c --enable-libvorbis --enable-libvpx --enable-libx264 --enable-libxavs --enable-
libxvid --enable-zlib
  libavutil      51. 34.100 / 51. 34.100
  libavcodec     53. 53.100 / 53. 53.100
  libavformat    53. 29.100 / 53. 29.100
  libavdevice    53.  4.100 / 53.  4.100
  libavfilter     2. 57.101 /  2. 57.101
  libswscale      2.  1.100 /  2.  1.100
  libswresample   0.  5.100 /  0.  5.100
  libpostproc    51.  2.100 / 51.  2.100
[sunrast @ 02645B60] invalid depth
[image2 @ 020E2320] Could not find codec parameters (Video: sunrast)
32.ras: could not find codec parameters

Attachments (3)

32.rar​ (1.3 MB) - added by ami_stuff 9 years ago.
patchsunrast32bpp.diff​ (578 bytes) - added by cehoyos 9 years ago.
lead_32bpp.7z​ (1.1 MB) - added by ami_stuff 6 years ago.

Change History (13)

Changed 9 years ago by ami_stuff

comment:1 Changed 9 years ago by cehoyos

  • Component changed from undetermined to avcodec
  • Keywords sunrast added
  • Priority changed from normal to wish
  • Reproduced by developer set
  • Status changed from new to open
  • Type changed from defect to enhancement
  • Version changed from unspecified to git-master

Do you think the file is meant to be completely transparent? (It is imo likely that it actually should not be transparent at all.)
Could you try to encode an image that is part transparent, part opaque?

Changed 9 years ago by cehoyos

comment:2 Changed 9 years ago by ami_stuff

Sorry, I don't know how to create transparent sun raster (if it's possible), so I suggest support it without it.

Version 0, edited 9 years ago by ami_stuff (next)

comment:3 Changed 9 years ago by cehoyos

  • Blocked By set to 869

Mostly fixed, remaining issues depend on ticket #869.

comment:4 Changed 9 years ago by cehoyos

  • Blocked By changed from 869 to #869

comment:5 Changed 9 years ago by richardpl

IMHO blocking this bug by another, unrelated bug is pointless. This bug should and can be fixed without waiting for #869 to be fixed.

comment:6 Changed 9 years ago by cehoyos

  • Blocked By changed from #869 to 869
  • Resolution set to fixed
  • Status changed from open to closed

comment:7 Changed 9 years ago by richardpl

ImageMagick displays wrong colors for this sample, what application created this file?

comment:8 Changed 9 years ago by richardpl

  • Blocked By 869 deleted
  • Priority changed from wish to normal
  • Reproduced by developer unset
  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Type changed from enhancement to defect

Our decoder incorrectly decodes 32bit ras created with ImageMagick.

Need more info about generated sample.

comment:9 Changed 9 years ago by cehoyos

  • Resolution set to fixed
  • Status changed from reopened to closed

Attached sample decodes visually incorrect now, but decoder matches what Gimp, ImageMagick and xview do and what the terse specification indicates.

Changed 6 years ago by ami_stuff

comment:10 Changed 6 years ago by ami_stuff

one more sample

Note: See TracTickets for help on using tickets.