ccn-lite icon indicating copy to clipboard operation
ccn-lite copied to clipboard

code styling

Open mehlis opened this issue 12 years ago • 4 comments

the code is not formatted to a style, perhaps ccn-lite can adapt the riot style: https://github.com/RIOT-OS/RIOT/blob/master/.astylerc

mehlis avatar Jul 28 '13 20:07 mehlis

On Sun, 28 Jul 2013, Christian Mehlis wrote:

the code is not formatted to a style, perhaps ccn-lite can adapt the riot style: https://github.com/RIOT-OS/RIOT/blob/master/.astylerc

true.

Style violations are even deliberate when it comes to ccnl-core.c otherwise I could hardly fit the code into the magic 1000 lines. I agree that at some point more "santity" (and more than 1000 lines) will have to come, but lets be liberal for the time being.

The suggested RIOT astylerc looks nice and very readable, I agree. And one also sees that ccn-lite needs a lot more comments...

thanks, cft

tschudin avatar Jul 29 '13 11:07 tschudin

I'll try to find a non bloating style. perhaps reducing the loc a bit!

mehlis avatar Jul 29 '13 12:07 mehlis

On Mon, 29 Jul 2013, Christian Mehlis wrote:

I'll try to find a non bloating style. perhaps reducing the loc a bit!

new functionality first ;-), but yes, as a pastime, why not? have fun, cft

tschudin avatar Jul 29 '13 14:07 tschudin

@tschudin Don't worry about it, "less than 2000 lines of code" still sounds impressive.

felixrabe avatar Jun 16 '14 13:06 felixrabe