Skip to content
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

Fix #1404 - fake native class throws an ugly exception #1490

Merged
merged 5 commits into from
Sep 13, 2018

Conversation

fdodino
Copy link
Collaborator

@fdodino fdodino commented Sep 5, 2018

Acá hice dos cosas, por un lado mejoré el mensaje de error cuando el class loader quiere cargar una clase y no existe (también agregué un test), y por otra parte el stack trace de Java en la consola es algo que causa muy mala impresión, la anulé por el momento. La mayoría de las veces resolvemos esas cuestiones replicando el incidente por lo que el stack trace de java con código xtend es poco útil.

image

@fdodino fdodino added this to the Wollok v1.7.1 milestone Sep 5, 2018
@ghost ghost assigned fdodino Sep 5, 2018
@ghost ghost added the in progress label Sep 5, 2018
@coveralls
Copy link

coveralls commented Sep 5, 2018

Coverage Status

Coverage increased (+0.008%) to 90.037% when pulling 00f60e3 on fix-#1404-native-fake-class into f674313 on dev.

@npasserini
Copy link
Member

npasserini commented Sep 5, 2018 via email

@fdodino
Copy link
Collaborator Author

fdodino commented Sep 6, 2018

Sí, a mí también me sonaba raro y hablando con @Juancete recién él se los había sacado (está laburando por otro issue similar de los programas). De acuerdo entonces con sacar el histórico WVM Error...

@fdodino
Copy link
Collaborator Author

fdodino commented Sep 7, 2018

Juan me pasó lo que había estado laburando, y ahora me parece que quedó bastante limpio:

image

Voy a hacer algunas pruebas más y lo subo.

@fdodino
Copy link
Collaborator Author

fdodino commented Sep 7, 2018

Pruebas de #1379

image

image

fdodino added a commit that referenced this pull request Sep 7, 2018
@Juancete
Copy link
Contributor

Juancete commented Sep 7, 2018

👍 Excelente!!!

@fdodino
Copy link
Collaborator Author

fdodino commented Sep 7, 2018

Ahí le metí un último toque, probando hacer en un test

	test "2" {
		assert.that(2)
	}

Tiraba "ERROR: null"

Así que modifiqué los errores del intérprete para que busque la causa, y si es una excepción de Wollok se muestre como tal

image

image

@fdodino
Copy link
Collaborator Author

fdodino commented Sep 8, 2018

@npasserini listo para mergear si tiene tu ok

@@ -10,5 +10,7 @@ class GameboardConfiguration extends LwjglApplicationConfiguration {
this.title = gameboard.title
this.width = gameboard.pixelWidth()
this.height = gameboard.pixelHeight()

System.setProperty("org.lwjgl.opengl.Display.allowSoftwareOpenGL", "true");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

¿Esto lo queremos mergear?

Salvo que lo hayas metido intencionalmente, probablemente esto es culpa mía, yo lo había metido en dev para probarlo en el product. Confirmame y yo me ocupo de deshacerlo.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sí, pensé que podría solucionarle a algún pibe Wollok Game, pero si lo volás mejor..

@npasserini
Copy link
Member

npasserini commented Sep 13, 2018 via email

@npasserini npasserini merged commit df25941 into dev Sep 13, 2018
@ghost ghost removed the in progress label Sep 13, 2018
@fdodino fdodino deleted the fix-#1404-native-fake-class branch June 21, 2019 21:44
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.

None yet

4 participants