search

sisoputnfrba / so-commons-library

TADs de uso comun en aplicaciones desarrolladas en C
http://sisoputnfrba.github.io/so-commons-library/
GNU General Public License v3.0
106 stars 174 forks source link

Agrego #mem_hexstring y #mem_hexdump #101

Closed tferraro closed 6 years ago

tferraro commented 6 years ago

Resolves #91

gastonprieto commented 6 years ago

tests?

tferraro commented 6 years ago

Los tengo en proceso, subí esto así lo podemos ir viendo por si hay algo que no cierra. No es precisamente una fiesta comprar contra un string, así que estoy viendo como hacerlos quedar más entendibles :stuck_out_tongue: .

gastonprieto commented 6 years ago

@tferraro queres que lo mergee? con un warning de beta (?)

tferraro commented 6 years ago

Si me das esta semana le hago tests, sino lo queda a criterio(?) jaja (colgue mal con esto)

El El dom, 26 ago. 2018 a las 17:35, Gaston Prieto notifications@github.com escribió:

@tferraro https://github.com/tferraro queres que lo mergee? con un warning de beta (?)

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/sisoputnfrba/so-commons-library/pull/101#issuecomment-416069987, or mute the thread https://github.com/notifications/unsubscribe-auth/AGx_i6Jj2k0wqwu2rkwfxq36eSuyNDUhks5uUwaRgaJpZM4TuH6J .

tferraro commented 6 years ago

Ahora si. @gastonprieto podés darle un review más lindo 😛

gastonprieto commented 6 years ago

me parece que tenes un leak porque a travis no le gusto :P

tferraro commented 6 years ago

@gastonprieto pensé que se había mergeado esto, ni me di cuenta hasta hoy. De paso al fix, le agregué un test más para mostrar que anda con no-strings 😄

gastonprieto commented 6 years ago

jaja, genial :heart:

Me voy a tener que dejar de hacer el boludo y revisarlo :p