Structural improvements for low-level NFC command. No new function.

[rdollet]

No description provided.

[bvernoux]

Warning this update does not build anymore with makefile:
multiple definition of `the_exception_context'

Please build it with both EmBlocks & makefile as the both are supported (official release is makefile).

Please also cleanup Warnings like "hydranfc\low_level\hydranfc_cmd_transparent.c:84:2: warning: 'return' with a value, in function returning void [enabled by default]" before to do any push.

Please before to do any push use "Format using AStyle" in EmBlocks with Linux Kernel rules on new files (it is not the case on hydranfc_low_microrl.c ...)

[rdollet]

Sorry for the mess. I thought you were reviewing the pull request before merging.
I'm not using makefiles, that was the reason I wanted to use EmBlocks. So even if I edit those, it's very likely I will not be able to test.
I've setup the AStyle formatting. I didn't pay attention that it was not enforced automatically.
I've fixed the return. my bad.

Is there a way fo your to test the pull before merging ?

[bvernoux]

Maybe the best is you create a branch (for big modifications WIP) on your own repository and then I could check locally if all is ok and you can pull that branch on the trunk.
For small modifications you can directly do pull like now (when you have built it correctly and tested it before).

[rdollet]

Ok, mais il y aura toujours le risque que les makefiles soient foireux si tu ne valides pas.
Au passage il y a un truc bizarre avec le semi-hosting. Printf se plante quand il y a des paramètres. J'ai rien trouvé sur le web, donc c'est sûrement un pb dans mon setup.
Au passage, j'ai introduit la syntax try/catch. Cela devait être pratique
A+
Richard

Sent from my iPad Mini

On Nov 16, 2014, at 8:32 AM, Benjamin Vernoux [email protected] wrote:

Maybe the best is you create a branch (for big modifications WIP) on your own repository and then I could check locally if all is ok and you can pull that branch on the trunk.
For small modifications you can directly do pull like now (when you have built it correctly and tested it before).

—
Reply to this email directly or view it on GitHub.

[doegox]

D'expérience utiliser l'interface github pour accepter un patch n'est pas idéal, personnellement j'interprète un pull req. comme une simple invitation à faire un merge "à la main". => je prends la branche de l'autre développeur chez moi, je teste et je merge, localement. Mais faire un merge par un simple clic sur une interface web, non merci.

[bvernoux]

Oui tout a fait, c'est aussi a moi de faire l'effort, j'ai trop l'habitude
qu'un pull est final (compilé et testé un minimum)

L'idéal c'est ausis d'en parler sur IRC channel #hydrabus sur freenode

2014-11-16 19:19 GMT+01:00 Philippe Teuwen [email protected]:

D'expérience utiliser l'interface github pour accepter un patch n'est pas
idéal, personnellement j'interprète un pull req. comme une simple
invitation à faire un merge "à la main". => je prends la branche de l'autre
développeur chez moi, je teste et je merge, localement. Mais faire un merge
par un simple clic sur une interface web, non merci.

—
Reply to this email directly or view it on GitHub
#8 (comment).

[rdollet]

Cela ne me derange pas d'essayer de mettre a jour les makefiles et d'avoir une branche séparée, si cela permet d'améliorer les choses. C'est un beau projet, autant pas tout casser :-) C'est juste que le merge aveugle sur le 'main' me parait pas non plus ideal. A+

[bvernoux]

Ce n'est pas un problème pour les makefile il faut juste m'indiquer dans le
commit que c'est pas a jour.
Le but c'est d'avoir le minimum de contrainte.

2014-11-17 16:24 GMT+01:00 rdollet [email protected]:

Cela ne me derange pas d'essayer de mettre a jour les makefiles et d'avoir
une branche séparée, si cela permet d'améliorer les choses. C'est un beau
projet, autant pas tout casser :-) C'est juste que le merge aveugle sur le
'main' me parait pas non plus ideal. A+

—
Reply to this email directly or view it on GitHub
#8 (comment).

[bvernoux]

There is a critical error: the exception do multiple definition during link with native makefile build with GNU_ARM_4_7_2013q3
I have removed the exception management until it is fixed.