Skip to content

Properly fix DisplayMetrics.DENSITY_DEVICE_STABLE mapping #1084

Description

@jonpryor

Commit ef77024 manually removed the DisplayMetrics.DENSITY_DEVICE_STABLE enum binding, added by commit 50c0c15. This fix is incomplete/incorrect: the problem is that the underlying tooling in build-tools/enumification-helpers is doing "something" wrong, which means this invalid mapping will be reintroduced in the future.

We need to fix build-tools/enumification-helpers so that DisplayMetrics.DENSITY_DEVICE_STABLE isn't reintroduced for enum mapping.

Activity

  1. added this to the d15-6 milestone on Dec 7, 2017
  2. atsushieno commented on Dec 7, 2017

    @atsushieno
    Contributor

    And someone needs to investigate why the PR build was green, which is why I think it's fine and it's merged.

  3. modified the milestones: d15-6, d15-7 on Jan 4, 2018
  4. atsushieno commented on Jan 26, 2018

    @atsushieno
    Contributor

    enumification-helpers is fixed, and we have already discovered that api-compatibility didn't run inter-api-diff and therefore this kind of failure didn't get detected. And there is already 6dfba92 so nothing needs to be tracked from this issue anymore. Closing.

  5. ghost locked as resolved and limited conversation to collaborators on Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions