FR: support @MainThread on fields [345674558]

Assigned

Feature Request

Status Update

No update yet.

Description

ub...@gmail.com

created issue #1

Jun 7, 2024 03:15PM

Component used: Annotation Version used: 1.8.0 Devices/Android versions reproduced on: N/A

I would like to see @MainThread targets include AnnotationTarget.FIELD, to express a declarative contract that a field should only be accessed from main thread:

class Foo {
    @MainThread
    var bar: Int = 0
}

Comments

ub...@gmail.com <ub...@gmail.com> #2Jun 7, 2024 03:16PM

Same for @UiThread, @AnyThread, @BinderThread I suppose.

Message last modified on Jun 7, 2024 03:18PM

ub...@gmail.com <ub...@gmail.com> #3Jun 7, 2024 03:21PM

It might actually be an oversight/bug, rather than a new feature.

Message last modified on Jun 7, 2024 03:22PM

ub...@gmail.com <ub...@gmail.com> #4Jun 7, 2024 03:31PM

I'm not sure if what should be added is @AnnotationTarget.FIELD or @AnnotationTarget.PROPERTY, or if both might make sense.

I'd think it would also be appropriate to add @AnnotationTarget.LOCAL_VARIABLE.

al...@google.com <al...@google.com> Jun 10, 2024 02:43PM

Reassigned to tn...@google.com.

Issue 345674558

Description

Issue summary

Comments

ub...@gmail.com <ub...@gmail.com> #2Jun 7, 2024 03:16PM

ub...@gmail.com <ub...@gmail.com> #3Jun 7, 2024 03:21PM

ub...@gmail.com <ub...@gmail.com> #4Jun 7, 2024 03:31PM

al...@google.com <al...@google.com> Jun 10, 2024 02:43PM

Add comment

Issue metadata