After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 778160 - raw-load: Miscellaneous fixes and improvements
raw-load: Miscellaneous fixes and improvements
Status: RESOLVED FIXED
Product: GEGL
Classification: Other
Component: operations
git master
Other All
: Normal normal
: ---
Assigned To: Default Gegl Component Owner
Default Gegl Component Owner
Depends on:
Blocks:
 
 
Reported: 2017-02-04 00:57 UTC by Debarshi Ray
Modified: 2017-02-07 09:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
operations/external/raw-load: Use a named constant, not a magic number (894 bytes, patch)
2017-02-04 01:18 UTC, Debarshi Ray
committed Details | Review
operations/external/raw-load: Say NO to OOM safety (1.30 KB, patch)
2017-02-04 01:18 UTC, Debarshi Ray
committed Details | Review
operations/external/raw-load: Remove redundant code (788 bytes, patch)
2017-02-04 01:19 UTC, Debarshi Ray
none Details | Review
operations/external/raw-load: Set user_data as soon as it is allocated (1.20 KB, patch)
2017-02-04 01:19 UTC, Debarshi Ray
committed Details | Review
operations/external/raw-load: Reinitialize libraw upon path change (1.03 KB, patch)
2017-02-04 01:20 UTC, Debarshi Ray
committed Details | Review
operations/external/raw-load: Be more assertive about what we expect (1.11 KB, patch)
2017-02-04 01:20 UTC, Debarshi Ray
committed Details | Review
operations/external/raw-load: Remove redundant code (1.02 KB, patch)
2017-02-04 17:23 UTC, Debarshi Ray
committed Details | Review
operations/external/raw-load: Don't leak the cached path (830 bytes, patch)
2017-02-04 21:42 UTC, Debarshi Ray
committed Details | Review

Description Debarshi Ray 2017-02-04 00:57:33 UTC
The libraw-based RAW loader has various small bugs and has some room for improvement.
Comment 1 Debarshi Ray 2017-02-04 01:18:22 UTC
Created attachment 344896 [details] [review]
operations/external/raw-load: Use a named constant, not a magic number
Comment 2 Debarshi Ray 2017-02-04 01:18:40 UTC
Created attachment 344897 [details] [review]
operations/external/raw-load: Say NO to OOM safety
Comment 3 Debarshi Ray 2017-02-04 01:19:00 UTC
Created attachment 344898 [details] [review]
operations/external/raw-load: Remove redundant code
Comment 4 Debarshi Ray 2017-02-04 01:19:32 UTC
Created attachment 344899 [details] [review]
operations/external/raw-load: Set user_data as soon as it is allocated
Comment 5 Debarshi Ray 2017-02-04 01:20:03 UTC
Created attachment 344900 [details] [review]
operations/external/raw-load: Reinitialize libraw upon path change
Comment 6 Debarshi Ray 2017-02-04 01:20:25 UTC
Created attachment 344901 [details] [review]
operations/external/raw-load: Be more assertive about what we expect
Comment 7 Debarshi Ray 2017-02-04 17:23:00 UTC
Created attachment 344941 [details] [review]
operations/external/raw-load: Remove redundant code
Comment 8 Debarshi Ray 2017-02-04 21:42:13 UTC
Created attachment 344951 [details] [review]
operations/external/raw-load: Don't leak the cached path