On 2015-02-20 19:02, Uwe Hermann wrote: > > On Tue, Feb 17, 2015 at 01:16:54AM +0100, Aurelien Jacobs wrote: >>> If present, add serial to description for better identification >> >> Appart from the spaces instead of tabs indentation, usage of snprintf() >> is slightly wrong: >> snprintf(description, sizeof(description)-1, ..... >> the -1 is useless and misleading. > > Merged with small indentation fixes as mentioned, thanks! I also dropped all > the "-1" from all snprintf() invocations as that's indeed a bit misleading. >
Ah my fault - I had strncat() in mind when writing those statements. As for the indentation - also my fault - my editor was setup for $work standards - and those are unfortunately the opposite of the sigrok project standards. Thanks :-) /Uffe ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ sigrok-devel mailing list sigrok-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/sigrok-devel
