• Status Closed
  • Percent Complete
  • Task Type Bugs
  • Category Utils
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Version 3.3
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by fip - 2009-08-08
Last edited by bertrik - 2009-08-09

FS#10496 - read_image close file?

I am not sure about whether it is actually used somewhere, but shouldn't
read_image_file_offset also close the file?

Closed by  bertrik
2009-08-09 22:21
Reason for closing:  Fixed
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Function containing the bug has been removed in svn r22232, thanks for reporting.

I think your concern is valid, I can't see these files being closed anywhere.
Your patch will not work because the new code can not be reached.

The attached patch should fix that.

Hmm, on closer inspection, nobody seems to use that function so it's not causing any problems and we might just remove it altogether, see attached patch.
Maybe we should ask the original author first before removing.


Available keyboard shortcuts


Task Details

Task Editing