[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