It was a quick attempt at showing the changes not needed see http://linuxcnc.org/docs/html/code/Style_Guide.html the first paragraph about do no harm
Most cannot help with the docs because you understand the changes not us so again some indication what the changes do and what others need to know, needs to come from you so others can help update any docs. Dave Caroline On 08/03/2015, Slavko Kocjancic <[email protected]> wrote: > On 08. 03. 2015 09:38, Dave Caroline wrote: >> With a diff program or better a merge program like meld one can easily >> see the white space changes that are extra to the real code changes >> and can confuse or waste time of a reviewer. Whitespace changes are >> often made by badly set editorr or a feeling that formatting may not >> be as you wish. >> >> I downloaded streamer.c from git and the new version and created two >> files from them >> >> first I used meld to fix whitespace differences >> this created >> http://www.collection.archivist.info/archive/mirror/linuxcnc/streamer.c.slv.djc >> I exported a diff patch >> http://www.collection.archivist.info/archive/mirror/linuxcnc/streamer.dif >> >> Dave Caroline >> > > Hello... > Now I'm confused even more. The c.slv.djc file has (in geanny) wery > strange whitespace. And even more there seems to be mix of tabs and > multiple spaces even in same line. But the .dif seems to be just right. > So thanks for that update. > ... And my editor is set to use tab char (4) instead in space. So > probably that affect the code. > > Slavko. > > > ------------------------------------------------------------------------------ > 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/ > _______________________________________________ > Emc-developers mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/emc-developers > ------------------------------------------------------------------------------ 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/ _______________________________________________ Emc-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/emc-developers
