no debería porque los parámetros tienen prefijo _ y las variables lo tienen en el sufijo. Sería error del compilador. Hágale push al código para revisarlo porfa.
No, debería haber pero yo lo olvidé 
Claudio Viquez|@cviquezc:
Y hay como un templete para el PR? Es que el link de contribution me tira al wiki donde hay como guidelines generales… o no sé si estoy buscando en el lugar equivocado
Cuando propone el PR le dice algo algunas cosas. Nada muy relevante para esto.
Claudio Viquez|cviquezc:
Listo, https://github.com/OpenZeppelin/openzeppelin-solidity/pull/1171
Durante el día mi tiempo de respuesta puede no ser muy bueno por el brete por cualquier cosa
Claudio Viquez|@cviquezc:
Mae @elopio todavía puedo entrarle a lo de sustituir las variables públicas por internas con getters en OZ?
Claudio sí, ese está aquí: https://github.com/OpenZeppelin/openzeppelin-solidity/issues/1174
Sólo tiene que comentar diciendo que lo va a hacer. Pero yo le recomendaría hacer un detour para saltar a ver algo de javascript primero.
Que le parece este para empezar: https://github.com/OpenZeppelin/openzeppelin-solidity/issues/1180 ?
Claudio Viquez|cviquezc:
Suena bien 
voy a echarle un ojo a los otros unittests para ver más o menos cómo está la cosa
Tuanis
…
Hemos estado trabajando un montón con el compilador de solidity. Si les interesa aprender sobre el ensamblador que ejecuta la máquina virtual de ethereum, estos posts les van a servir: https://blog.zeppelin.solutions/deconstructing-a-solidity-contract-part-i-introduction-832efd2d7737
@Claudio tengo más ideas de PRs, aviseme cuando quiera seguir.
Claudio Viquez|@cviquezc:
Cuando quiera le puedo entrar a otro…
@Claudio para terminar el tour, antes de entrarle a más refactor, creo que estaría bueno hacer algo más desde 0. Puede ser una regla de solium como esta: https://github.com/OpenZeppelin/solium-plugin-zeppelin/pull/30/files
Pero para lo que hizo antes de trailing underscore.
Claudio Viquez|@cviquezc:
@elopio como tema aparte, estoy tratando de entender FairCoin… no entiendo bien “proof of cooperation”, los nodos se ponen de acuerdo para elegir quien va a ser el siguiente nodo en minar un bloque?
ajá. Es un round robin, no una competencia de minería.
Claudio Viquez|@cviquezc:
ok ahora más tarde le echo un ojo
Tiene mucho sentido, porque la minería en otras cryptos (la mayoría) es ineficiente (y a mi criterio, un gasto de recursos computacionales)
Claudio Viquez|@cviquezc:
pero si es un round robin, significa que hay un concepto de orden? y entonces los nuevos nodos se agregan de últimos en alguna cola?
um, diay, es circular. No hay últimos, pero sí, los nodos nuevos entran a algún lugar del círculo, y eventualmente les toca antes de que finalice una vuelta.
@Claudio instálese la app de faircoin de la tienda de aplicaciones y le mando unos cuantos para que pruebe.
Claudio Viquez|@cviquezc:
Qué pasa si el siguiente bloque me toca a mí y me desconecto o no mino el bloque? Hay un tiempo máximo que me da la red para minar el bloque?
Esta parte es buenísima: https://chain.fair.to/address?address=fGDa5sk5xBaoN1npV92bS8Waz22rKLX7T6
Esa es la dirección del monedero de mi nodo. Ya lleva como 6 meses arriba, y apenas tiene 1 faircoin ganado. No alcanza ni para la luz, entonces el incentivo para colaborar con la red no es económico.
sí, lo toma el nodo siguiente.