[Jderobot] Coding style

Maykel Moya mmoya en mmoya.org
Dom Ene 26 00:08:13 CET 2014


Hola

Hace un tiempo tuve un intercambio con Roca a partir de un comentario
que le hice respecto a tener en el proyecto una guía de estilo. Él me
pidió mandar un mensaje a la lista.

Entre las tantas ventajas que aportaría uniformar la forma de escribir
se encuentra disminuir la fricción a la hora de integrar/generar parches
en formato diff.

La guía de estilo de código para C++ de Google[1] es bastante popular.
Por ejemplo Gazebo la usa[2] y ROS la referencia[3]. Es la que propongo
que adoptemos.

Google tiene a disposición un linter[4]. Corriendo un lint contra el
trunk sin ningún filtro devuelve 43K errores, forzando a que sólo
chequee whitespace/tab (uno de los tantos chequeos disponibles) arroja
18K errores. Estos números son revisando *.h, *.c, *.cc y *.cpp dentro
de src/ y excluyendo src/testing y src/stable/interfaces.

Considero que un mínimo razonable es solucionar los whitespace/tab y
luego ir incorporando más chequeos al linter automático. Los
whitespaces/tab se pueden corregir automáticamente con: astyle
--style=linux --indent=spaces=2.

Roca, te adjunto el script que uso para ejecutar cpplint.py para que
valores integrarlo a los hooks del repo.

Saludos,
maykel

[1] https://google-styleguide.googlecode.com/svn/trunk/cppguide.xml
[2] https://gazebosim.org/user_guide/modifying__style.html
[3] http://wiki.ros.org/CppStyleGuide
[4] https://google-styleguide.googlecode.com/svn/trunk/cpplint/cpplint.py

------------ próxima parte ------------
Se ha borrado un mensaje que no está en formato texto plano...
Nombre     : runlinter.sh
Tipo       : application/x-shellscript
Tamaño     : 313 bytes
Descripción: no disponible
Url        : http://gsyc.escet.urjc.es/pipermail/jde-developers/attachments/20140126/22f64bd2/attachment.bin 


More information about the Jde-developers mailing list