Skip to content
Snippets Groups Projects

medialibrary: thumb: check fclose while saving

Merged Alaric Senat requested to merge asenat/vlc:medialib-wrapper into master
All threads resolved!

It should not be assumed that the thumbnail was saved properly when fclose fails. The RAII wrapper here does not improve readability or shorten the code.

This MR supersede !4057 (closed) by working around the warning and handling the error at the same time.

Merge request reports

Merge request pipeline #372226 passed

Merge request pipeline passed for f1d4ade9

Merged by Steve LhommeSteve Lhomme 1 year ago (Aug 9, 2023 1:49pm UTC)

Loading

Pipeline #372242 passed

Pipeline passed for f1d4ade9 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading