Hello Sean, Recently I had restarted working on Materials Database and had cleaned the code according to the guidelines in the Hacking file. But Hacking file is C / C++ centred and so for some PHP reference I saw the CIAnotify extension as it is on BRL-CAD official GitHub. I have observed the following differences which are contradiction of Hacking file, recommended to be kept in mind while contributing to BRL-CAD:
a) The code of CIAnotify[1] is indented by using a tab-key(8 characters) whereas Hacking file says to indent using 4 whitespaces and use tab for 8 characters. b) There is also a space after the parenthesis starts and before it ends(line no.29). example: if ( $cnTo == '' ) c) Then in line number 50 of [1], the curly brackets of a function start from the same line whereas Hacking file says it should begin in a new line in case of functions. d) Then "/" operator for comments is also used in [1] which is discouraged in Hacking file. e) Lastly how to indent HTML which is embedded in PHP, 4 characters or 8? Please help me with this confusion so that I can push perfect code on BRL-CAD GitHub according to the coding standards. [1] https://github.com/BRL-CAD/web/blob/master/w/extensions/CIAnotify/CIAnotify.php -- Thanks Albert www.coderalbert.wordpress.com https://github.com/albertcoder/ || If you are not pursuing your goal, you are literally committing spiritual SUICIDE. || ------------------------------------------------------------------------------ Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk _______________________________________________ BRL-CAD Developer mailing list brlcad-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/brlcad-devel
