Skip to content

entry point finding #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

entry point finding #22

wants to merge 1 commit into from

Conversation

lrytz
Copy link
Member

@lrytz lrytz commented Feb 14, 2014

not quite done yet, need to figure out how to best write
definitions.isArrayOfSymbol (see scalac). for that, need to
understand the Type classes in dotty.

review by @magarciaEPFL

that's the last contribution before my ski holidays - will be back to work on the backend on february 24

not quite done yet, need to figure out how to best write
`definitions.isArrayOfSymbol` (see scalac). for that, need to
understand the `Type` classes in dotty.
@lrytz
Copy link
Member Author

lrytz commented Feb 14, 2014

build failures are expected for now on the backend branch.

@VladUreche
Copy link
Contributor

@lrytz: sorry, Travis can't filter the PRs by target branch -- either all PRs are checked or none are. It won't build the backend branch though, so PRs are the only case when Travis will bother you.

@lrytz
Copy link
Member Author

lrytz commented Feb 14, 2014

no problem :)

@sjrd
Copy link
Member

sjrd commented Feb 14, 2014

Actually it will build all branches, indiscriminately. You can filter out branches "by hand" by using explicit tests in the script:

- if [[ "$TRAVIS_BRANCH" != "backend" ]]; then sbt ...; fi

It is also possible (and I recommend it), to disable builds for branches that do not have a .travis.yml file. Someone with the rights to do so should go to | Accounts (top-right corner of the page), go to the dotty repo, click on the settings icon, and enable "Build only commits with .travis.yml file".

@sjrd
Copy link
Member

sjrd commented Feb 14, 2014

Actually it will build all branches, indiscriminately.

Oh, my mistake. I did not realize you had "branches: only: master" in your .travis.yml.

@lrytz
Copy link
Member Author

lrytz commented Feb 28, 2014

ping @magarciaEPFL

@DarkDimius
Copy link
Contributor

subsumed by #124

@DarkDimius DarkDimius closed this May 6, 2014
OlivierBlanvillain pushed a commit to OlivierBlanvillain/dotty that referenced this pull request Dec 8, 2016
At issue is that the optimizer would eliminate closure classes
completely, then neglect to eliminate those classes from the
container's InnerClasses attribute. This breaks tooling which
expects those entries to correspond to real classes.

The code change is essentially mgarcia's - I minimized it and
put the caches in perRunCaches, and added the test case which
verifies that after being compiled under -optimise, there are
no inner classes. Before/after:

  7,8d6
  <   InnerClasses:
  <        public final scala#22; //class A_1$$anonfun$f$1
  37,45c35,40
  <   scala#21 = Utf8               A_1$$anonfun$f$1
  <   scala#22 = Class              scala#21            //  A_1$$anonfun$f$1
  <   scala#23 = Utf8               Code
  ---
  >   scala#21 = Utf8               Code
OlivierBlanvillain pushed a commit to OlivierBlanvillain/dotty that referenced this pull request Dec 12, 2016
At issue is that the optimizer would eliminate closure classes
completely, then neglect to eliminate those classes from the
container's InnerClasses attribute. This breaks tooling which
expects those entries to correspond to real classes.

The code change is essentially mgarcia's - I minimized it and
put the caches in perRunCaches, and added the test case which
verifies that after being compiled under -optimise, there are
no inner classes. Before/after:

  7,8d6
  <   InnerClasses:
  <        public final scala#22; //class A_1$$anonfun$f$1
  37,45c35,40
  <   scala#21 = Utf8               A_1$$anonfun$f$1
  <   scala#22 = Class              scala#21            //  A_1$$anonfun$f$1
  <   scala#23 = Utf8               Code
  ---
  >   scala#21 = Utf8               Code
pikinier20 added a commit that referenced this pull request Aug 16, 2022
szymon-rd added a commit that referenced this pull request Dec 9, 2022
WojciechMazur added a commit to WojciechMazur/dotty that referenced this pull request Mar 19, 2025
Backport "Fix CLA checks after domain change of CLA check server" to LTS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants