Files
Nikita Bobko d5da414213 Cleanup: code review fixes
https://jetbrains.team/p/ij/reviews/131139/timeline

The annotation name doesn't contain "Data"
    -            AddAnnotationFix.AddConsistentDataCopyVisibilityAnnotationFactory,
    +            AddAnnotationFix.AddConsistentCopyVisibilityAnnotationFactory,

I accidentially dropeed wrapping with <html> in 2e5b5df8c6e2830abc21af3f69ddca0a55435c9b.
This patch returns it back. Though most probably it doesn't matter.
The rendered text looks the same
    -        val htmlMessage = XmlStringUtil.escapeString(message).replace("\n", "<br>")
    +        val htmlMessage = XmlStringUtil.wrapInHtml(XmlStringUtil.escapeString(message).replace("\n", "<br>"))

It would be nice to emphasize the nature of those tests
(that they are not just about the private constructors in data classes,
but about the annotations/copy method consistency)
    rename from community/plugins/kotlin/idea/tests/testData/quickfix/dataClassPrivateConstructor
    rename to community/plugins/kotlin/idea/tests/testData/quickfix/dataClassConstructorVsCopyVisibility

GitOrigin-RevId: 11e996b008b288750ccdebb91f42f3c9dc843bf6
2024-04-16 11:58:29 +00:00
..