Messagekit: Images instantiated with `contentsOf:` leak their path

[yspreen]

There's two image inits in MessageKit which cause memory leaks:

/Sources/Models/MessageStyle.swift:85:        guard var image = UIImage(contentsOfFile: path) else { return nil }
/Sources/Extensions/UIImage+Extension.swift:39:        let image = UIImage(contentsOfFile: imagePath ?? "")

According to this answer on SO, using UIImage(data: Data(contentsOf: url)) instead fixes this issue.

[kinoroy]

Hey @yspreen thanks for reporting this! Would you be interested in submitting a PR for this? :) No pressure

[martinpucik]

@yspreen I think this is not used anymore on development branch. Could you please check if this could be consider fixed?

[kinoroy]

@martinpucik Confirmed, #1341 fixed this.