[ https://issues.apache.org/jira/browse/MSHADE-396?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17373880#comment-17373880 ]
Alexander Kriegisch edited comment on MSHADE-396 at 7/3/21, 2:52 AM: --------------------------------------------------------------------- [~Reamer], thanks for the notification. I am going to take a look as soon as I have some free cycles. Not being a committer in this project, my review is not authoritative, though. BTW, I am happy that someone uses my source shading implementation. It was heuristic to begin with - a perfect solution would require full Java parsing and probably some kind of source AST representation - and certainly has room for incremental improvement. This is the beauty of OSS development: There is always someone out there who can take your work one step further or sometimes even lift it to a whole new level. 🙂 was (Author: kriegaex): [~Reamer], thanks for the notification. I am going to take a look as soon as I have some free cycles. Not being a committer in this project, my review is not authoritative, though. BTW, I am happy that someone uses my source shading implementation. It was heuristic to begin with - a perfect solution would require full Java parsing and probable some kind of source AST representation - and certainly has room for incremental improvement. This is the beauty of OSS development: There is always someone out there who can your work one step further or sometimes even lift it to a whole new level. 🙂 > Improve SourceContent Shading > ----------------------------- > > Key: MSHADE-396 > URL: https://issues.apache.org/jira/browse/MSHADE-396 > Project: Maven Shade Plugin > Issue Type: Bug > Affects Versions: 3.2.4, 3.3.0 > Reporter: Philipp Dallig > Priority: Major > > MSHADE-252 has greatly improved the shading of the source content. I used > this version but found some problems in combination with a small pattern. > Examples: > We should exclude replacement in the following situations > - Pattern: "io" > Content: java.io.IOException; > Content: private String myException; > - Pattern: io > Content: private String ioInput > Content: String io, val; -- This message was sent by Atlassian Jira (v8.3.4#803005)