Uploaded image for project: 'JSword'
  1. JSword
  2. JS-128

AbstractSwordInstaller does not delete the temporary download file

    Details

      Description

      Maybe I am doing something wrong but I was having a lot of temporary swd files left on my Android sdcard so I think the temp file should be deleted in a finally block in AbstractSwordInstaller.run()

      + URI temp = null;
      try

      { // main code skipped }

      finally {
      job.done();
      + // tidy up after ourselves
      + if (temp != null) {
      + try

      { + NetUtil.delete(temp); + }

      catch (Exception e)

      { + log.warn("Error deleting temp download file:"+e.getMessage()); + }

      + }
      }

        Attachments

          Activity

            People

            • Assignee:
              dmsmith DM Smith
              Reporter:
              mjdenham Martin Denham
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: