search

ciaa / firmware_v1

Firmware de la CIAA
http://www.proyecto-ciaa.com.ar
129 stars 120 forks source link

#435 fix rtcs templates #436

Closed diegoVom closed 8 years ago

diegoVom commented 8 years ago

There are non supported symbols with the new Oil generator. The rtcs module and its example work now.

cpantel commented 8 years ago

I do not understand why there are those <?php ?> at the beginning of some lines.

cpantel commented 8 years ago

which symbols are not supported?

diegoVom commented 8 years ago

Los simbolos "<? ?>" estaban para tabular el template y de esa forma intentar mejorar la visibilidad y lectura del mismo.

Luego, el generador debia eliminarlos puesto que dichos tabs no son deseados en el fichero en C.

Con el nuevo generador esto no sucedía, los dejaba como estaba y luego el compilador de C tiraba error. Los cambié por "<?php ?>" para que se mantenga el tab en el template y desaparezca en el fichero generado.

mabeett commented 8 years ago

This patch could solve #435 .

@mcerdeiro @gmuro If you are considering not accepting this patch, please evaluate reverting the patches which generate #435 , or add an #error macro preprocessor statement, since now the example 'rtcs_example' builds fails.