Project

General

Profile

Bug #11355

rawspeed getCamera() with prefix-search issue

Added by Roman Lebedev over 2 years ago. Updated about 2 years ago.

Status:
Fixed
Priority:
Medium
Assignee:
Category:
General
Target version:
Start date:
12/12/2016
Due date:
12/12/2016
% Done:

100%

Affected Version:
git master branch
System:
all
bitness:
64-bit
hardware architecture:
amd64/x86

Description

https://github.com/klauspost/rawspeed/issues/166

I added that api in https://github.com/klauspost/rawspeed/commit/a45724ec3c201b67f5ff732317921c0feca878c6 and https://github.com/klauspost/rawspeed/commit/e1b5ccae3515773755621cee305170fa5ea63808.
And now i have just discovered how it backfires.
With raw file from https://redmine.darktable.org/issues/11354 from `Canon EOS Rebel T6`, which currently has broken alias, it matches to `Canon EOS Rebel T6i`.
That happens because internally "make model mode" is a string, https://github.com/klauspost/rawspeed/blob/a45724ec3c201b67f5ff732317921c0feca878c6/RawSpeed/CameraMetaData.h#L37


Related issues

Follows darktable - Camera Support #11354: Canon Rebel 6t, "Failed to read camera white balance information" Fixed 12/09/2016
Follows darktable - Bug #11108: Cameras.xml name "de-"mangling is broken, especially for nikon Fixed 08/10/2016

Associated revisions

Revision f41d2eb8
Added by Roman Lebedev about 2 years ago

RawspeedL CameraMetaData: make id a struct, fix search. Fixes #11355

Revision 622e6c1c
Added by Roman Lebedev about 2 years ago

Rawspeed submodule update. Fixes #11355

History

#1 Updated by Roman Lebedev over 2 years ago

  • Follows Camera Support #11354: Canon Rebel 6t, "Failed to read camera white balance information" added
  • Due date set to 12/12/2016
  • Start date changed from 12/09/2016 to 12/12/2016

#2 Updated by Roman Lebedev over 2 years ago

  • Follows Bug #11108: Cameras.xml name "de-"mangling is broken, especially for nikon added

#3 Updated by Roman Lebedev over 2 years ago

  • Target version changed from 2.2.0 to 2.4.0

#4 Updated by Roman Lebedev about 2 years ago

  • % Done changed from 0 to 100
  • Status changed from New to Fixed

Also available in: Atom PDF