Skip to content

added Dumb and DumbNoMore packet classes #1

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

Open
wants to merge 46 commits into
base: master
Choose a base branch
from
Open

Conversation

mer23
Copy link

@mer23 mer23 commented Apr 26, 2014

No description provided.

@itirabasso
Copy link
Contributor

You should use spaces instead of tabs.

}
// Put the packet id before the packet itself.
buffer.put(packetsMap.get(packet.getClass()).byteValue());
packet.write(buffer);
Copy link
Contributor

Choose a reason for hiding this comment

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

Se podría evitar esos paquetes que literalmente no hacen nada chequeando packet != null y dejarle el null en el enum.

MER added 18 commits May 1, 2014 22:44
…ing. Added MeditateTogglePacket in order to do so
…face UserCharacter in order to achieve character's login. For the same purpose made some changes and completions to several classes involved in a newly created character's first login
…at's currently logged in, if any. Couldn't find another way to gain access to the actual character (Connection's User is not the logged character)from incoming packet's classes. Now I can call connection.getUser().getAccount.getLoggedCharacter() and work with it from within the IncomingPacket.handle method. Updated only LoginNewCharacterPacket and LoginExistingCharacterPacket to suit these changes(Have to do so with the other IncomingPackets too).
…s. Now LoginNewCharacterPacketTest loads the maps. Now UserDAOIni sets user position both at create and load methods.
@jsotuyod
Copy link
Contributor

No puedo creer que nunca ví este PR... no me llegó alerta alguna por mail. Ante todo disculpas.

Hicist eun laburazo, pero temo que ahora, más allá de que no mergea por mis últimos cambios, terminaron habiendo demasiados cambios juntos para mirarlos / meterlos (46 commits son un poco mucho).

Me encantaría si pudieras hacer un montón de PRs mucho más chicos, en especial lo que ya hiciste con los paquetes (considerando que por lo que veo muchos ya están en commits independientes que pueden llevarse a un PR fácil).

Sino, intentaré tomarlos yo a mano.

@mer23
Copy link
Author

mer23 commented Mar 1, 2015

Hola @jsotuyod ,
Cloné el repo para arrancar pero eclipse no me está reconociendo los paths relativos. (no encuentra "resources/maps.properties" por ejemplo). Probé varias cosas pero nada funciona. Alguna idea?

@jsotuyod
Copy link
Contributor

jsotuyod commented Mar 2, 2015

Puede ser que tengas los proyectos importados mal / con otra raíz.

Borrá en Eclipse los proyectos, sin borrar los archivos de disco. Luego parate en la raíz de tu clone y hacé desde consola mvn eclipse:clean eclipse:eclipse. Finalmente, andá en Eclipse a File -> Import... -> Existing project into workspace; y dentro de la carpeta que clonaste debieras poder ver 2 .project para importar.

Ahí debieras salir jugando, con todas las dependencias y propiedades bien configuradas. Todos los binarios se ejecutarán con current working directory en la raíz de cada proyecto por default, con lo que encontrarán los resources externos al binario.

Cualquier cosa chiflame

@mer23
Copy link
Author

mer23 commented Mar 2, 2015

Listo! gracias ;)

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.

3 participants