Libgdx new version

[fdodino]

Contiene varios features de Wollok Game que no son retrocompatibles.

• WollokGame: upgrade lib gdx #1194 Se actualizó la librería libgdx a 1.9.8, el motor de gráficos sigue siendo el mismo (ellos no lo actualizaron a la 3)
• Divide Wollok Game in other Lib #1161 Position y Game ahora están en la biblioteca game.wlk (hay que importarlos)
• Adding sounds to Wollok Game #1452 agregamos sonido a Game
• onTick - Way to remove it #1475 onTick obliga a referenciar el listener por nombre , para luego poder borrarlo
• Add inmutable positions to lib #1318 y Implementation of WPosition #1152 Position es un objeto inmutable (eso fuerza a modificar todos los ejemplos de Wollok Game). Se actualizó el WollokDoc también para que refleje eso.
• Traducir métodos de game al inglés #1483 Se elimina la posibilidad de definir los métodos posicion() o imagen() para los objetos visuales, va solo la versión en inglés (se actualizó también el wollokDoc)
• antes se validaba que no cayera fuera del gameboard los objetos visuales que usaban una instancia de Position. Ahora eso es responsabilidad del programador, lo que sí garantiza WollokGame es que un personaje (Visual Character) no se va a mover fuera del tablero cuando te muevas con las flechas. Esto lo hice porque quizás en algún momento algún juego podría requerir un feature de salir del tablero, o bien esconder un objeto. Habría que ver qué pasa si vos querés que un personaje al salir de uno de los costados aparezca en otro sector del tablero (como en el Pacman). Pero el refactor lo corto acá para poder seguir con otros temas.

No deberíamos mergear hasta liberar la versión 1.7.2 o bien si estamos seguros que ya no vamos a hacer releases por el resto de la cursada 2018

[coveralls]

Coverage increased (+0.1%) to 90.183% when pulling ab683fb on libgdx-new-version into df25941 on dev.

[npasserini]

Dejé una duda picando ahí, pero da para charlar y bien podría resolverse en otro PR si querés mergear este.

La otra cosa que me queda picando para charlar, es si está bien que la forma de cancelar un listener está bien que sea un String. Otra opción que vi por ahí, es que el propio fwk devuelva un objeto que sirve para cancelar. Eso evita que yo tenga que estar construyendo strings que garanticen unicidad.

[npasserini]

Esto me da dudas, lo que yo entiendo es que si estoy en el borde del tablero y hago up me da la misma posición. ¿No sería más prolijo tirar una excepción + tener métodos para saber si up existe? Me parece medio peligroso porque estos métodos no devuelven lo que dicen.

[npasserini]

Entiendo que el character necesita eso, pero ¿por qué no lo implementamos en character?

[fdodino]

Pasó mucho tiempo, pero recuerdo ir caminando por Florida y haber pensado exactamente eso, en dejar que Position se vaya del tablero y ya, porque el Listener de Character ya tiene una forma de no irse. Así que sí, lo voy a cambiar.

[npasserini]

De hecho, veo que character ya tiene esa validación.

[fdodino]

La idea original de Alf era que se identificaran con Strings, lo que me parece que tiene como contra respetar la unicidad. Por otra parte tendría que mirar el código para saber qué tan complejo se vuelve devolver el objeto (ya que el Listener se crea del lado de Java), el string lo vuelve más fácil de usar desde diferentes contextos. Yo por ahora voy a hacer el toque de Position, yo mergearía y esperaría a alguna reunión donde podamos seguirla.