Tomek CEDRO wrote: > 1. Prefix all variables and function names with "oocd_". .. > this will dramatically increase code readability and debug
It seems to me that it only adds noise. > 2. Use layer name as second argument in function/variable names - > i.e. "oocd_interface_" for everything related to Interface layer, > "oocd_transport_" for everything related to transport layer, > "oocd_target_" for everything related to target layer, etc. Notice how oocd_ adds nothing of value to those names when they are listed next to each other. But I completely agree with using representative names for the constructs in the program, struct interface *, as an example. But adding noise at the start of names really makes no sense to me. The use case for such a prefix is when creating a library with a public API, which OpenOCD is not, and will not be in foreseeable time. //Peter ------------------------------------------------------------------------------ Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft MVPs and experts. ON SALE this month only -- learn more at: http://p.sf.net/sfu/learnnow-d2d _______________________________________________ OpenOCD-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openocd-devel
