Qin Chuanqing opened SPR-15748 and commented

I do work in nokia, we do a much strict standard for code which analysis by sonar, with the plugin of called sonarLint which can install in IDEA, I can find some bad code smell, an example is in org.springframework.util.FileSystemUtils, we can find below in attachment, I think we should do something for it. and will never introduce any more.

Affects: 4.3.9

Attachments:

• Screen Shot 2017-07-08 at 18.00.22.png (78.18 kB)

Issue Links:

• StandardMultipartFile.transferTo should fall back to manual copy if Part.write doesn't support absolute locations (e.g. on Jetty) [SPR-15257] #19822 StandardMultipartFile.transferTo should fall back to manual copy if Part.write doesn't support absolute locations (e.g. on Jetty)
• Expose Channel on Resource [SPR-14698] #19262 Expose Channel on Resource
• FileSystemUtils.deleteRecursively always returns false [SPR-15846] #20401 FileSystemUtils.deleteRecursively always returns false
• PropertySourcesPlaceholderConfigurer can not ignore resource if not found [SPR-16334] #20881 PropertySourcesPlaceholderConfigurer can not ignore resource if not found
• java.nio.file.Path support in FileSystemResource (with regular createRelative behavior, superseding PathResource) [SPR-16833] #21373 java.nio.file.Path support in FileSystemResource (with regular createRelative behavior, superseding PathResource)

Referenced from: commits 3714e7b, 12114a9