<feed xmlns='http://www.w3.org/2005/Atom'>
<title>github/Remap.git/src/test/kotlin/com/replaymod/gradle/remap/util, branch bettererror</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<link rel='alternate' type='text/html' href='https://git.nea.moe/github/Remap.git/'/>
<entry>
<title>Allow JDK_HOME to be set explicitly, required for Kotlin 1.8+</title>
<updated>2023-05-26T14:17:29+00:00</updated>
<author>
<name>Jonas Herzig</name>
<email>jonas@spark-squared.com</email>
</author>
<published>2023-05-26T14:15:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.nea.moe/github/Remap.git/commit/?id=3a12dc22d6ec3924067c7b32582cb6a1ed113344'/>
<id>3a12dc22d6ec3924067c7b32582cb6a1ed113344</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Add support for adding missing and removing unused imports</title>
<updated>2022-05-27T06:46:01+00:00</updated>
<author>
<name>Jonas Herzig</name>
<email>me@johni0702.de</email>
</author>
<published>2022-05-27T05:39:18+00:00</published>
<link rel='alternate' type='text/html' href='https://git.nea.moe/github/Remap.git/commit/?id=cde89808e3d9730fe784597bd6bbfc51753663a1'/>
<id>cde89808e3d9730fe784597bd6bbfc51753663a1</id>
<content type='text'>
This adds a post-process step which automatically adds unambiguous imports,
removes unused imports and sorts the import list (formatting matches standard
IntelliJ settings).
This will preserve line count across versions at all cost.

Java only for now because it's a lot more tricky with Kotlin and we don't yet
use Kotlin ourselves (and won't be preprocessing it in the future either).
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This adds a post-process step which automatically adds unambiguous imports,
removes unused imports and sorts the import list (formatting matches standard
IntelliJ settings).
This will preserve line count across versions at all cost.

Java only for now because it's a lot more tricky with Kotlin and we don't yet
use Kotlin ourselves (and won't be preprocessing it in the future either).
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix mapping of synthetic properties when overridden in Kotlin class</title>
<updated>2021-11-13T09:28:09+00:00</updated>
<author>
<name>Jonas Herzig</name>
<email>jonas@spark-squared.com</email>
</author>
<published>2021-11-12T11:58:49+00:00</published>
<link rel='alternate' type='text/html' href='https://git.nea.moe/github/Remap.git/commit/?id=3ee08cbce94a4e7e1ce668f06f5a612cd9f1e677'/>
<id>3ee08cbce94a4e7e1ce668f06f5a612cd9f1e677</id>
<content type='text'>
In these cases, the getMethod of the property will not have a Psi element and we
need to traverse up the overrides until we find one that does.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
In these cases, the getMethod of the property will not have a Psi element and we
need to traverse up the overrides until we find one that does.
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix synthetic property becoming shadowed by field of same name</title>
<updated>2021-11-12T14:23:25+00:00</updated>
<author>
<name>Jonas Herzig</name>
<email>jonas@spark-squared.com</email>
</author>
<published>2021-11-12T09:58:10+00:00</published>
<link rel='alternate' type='text/html' href='https://git.nea.moe/github/Remap.git/commit/?id=5723e6481d2f4e07b6820201e74924643b5687bc'/>
<id>5723e6481d2f4e07b6820201e74924643b5687bc</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix loss of changes when multiple target the same start point</title>
<updated>2021-11-12T14:23:25+00:00</updated>
<author>
<name>Jonas Herzig</name>
<email>jonas@spark-squared.com</email>
</author>
<published>2021-11-11T19:59:47+00:00</published>
<link rel='alternate' type='text/html' href='https://git.nea.moe/github/Remap.git/commit/?id=f1224518f86b0160149544dc8d56505a37a186e2'/>
<id>f1224518f86b0160149544dc8d56505a37a186e2</id>
<content type='text'>
While one might at first think that multiple changes should conflict if they
target that same start point, that is not necessarily true as long as no more
than one of them includes deletions: There may be an arbitrary number of
insertions at the same position (regular remapping never just inserts but
patterns can).
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
While one might at first think that multiple changes should conflict if they
target that same start point, that is not necessarily true as long as no more
than one of them includes deletions: There may be an arbitrary number of
insertions at the same position (regular remapping never just inserts but
patterns can).
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix remapping of qualified inner class reference in Kotlin code</title>
<updated>2021-11-11T20:38:47+00:00</updated>
<author>
<name>Jonas Herzig</name>
<email>jonas@spark-squared.com</email>
</author>
<published>2021-11-11T15:52:27+00:00</published>
<link rel='alternate' type='text/html' href='https://git.nea.moe/github/Remap.git/commit/?id=982982c63027e416bd78ca40708c7238e7eed26b'/>
<id>982982c63027e416bd78ca40708c7238e7eed26b</id>
<content type='text'>
When we used to remap `a.pkg.A.Inner` we would apply both mappings (the one for
the inner class and the one for the outer class) at the same time, resulting in
`b.pkg.B.Inner.Inner`.
The direct cause being that we only checked conflicts for the range of the
respective expression (which is just `A` and `Inner` for Kotlin, and therefore
not conflicting) instead of the parent as we should have.

With that fixed, it now also becomes apparent that we need to apply mappings for
dot qualified expressions back to front (otherwise the outer class takes
priority), so that is the second thing this commit change.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When we used to remap `a.pkg.A.Inner` we would apply both mappings (the one for
the inner class and the one for the outer class) at the same time, resulting in
`b.pkg.B.Inner.Inner`.
The direct cause being that we only checked conflicts for the range of the
respective expression (which is just `A` and `Inner` for Kotlin, and therefore
not conflicting) instead of the parent as we should have.

With that fixed, it now also becomes apparent that we need to apply mappings for
dot qualified expressions back to front (otherwise the outer class takes
priority), so that is the second thing this commit change.
</pre>
</div>
</content>
</entry>
<entry>
<title>Start adding tests</title>
<updated>2021-11-10T17:11:16+00:00</updated>
<author>
<name>Jonas Herzig</name>
<email>jonas@spark-squared.com</email>
</author>
<published>2021-11-10T08:34:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.nea.moe/github/Remap.git/commit/?id=7d62cd890cb44945e772e18cdb457b1173f4f853'/>
<id>7d62cd890cb44945e772e18cdb457b1173f4f853</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
</feed>
