<?xml version="1.0" encoding="UTF-8"?>
<rss version="2.0" xmlns:dc="http://purl.org/dc/elements/1.1/">
    <channel>
        <title>Remarks are  ; surely we should use //  or /* */</title>
        <description> Guys,

Going through the firmware code and it looks like the code is using ; as a remark.
Surely we should be following the C convention of // or /* and */.

; is a line terminator in C and PL/SQL

regards

Stephen


Here is the code in question in process_string


		  //newlines are ends of commands.
		  if (c != &#039;\n&#039;)
		  {
			// Start of comment - ignore any bytes received from now on
			if (c == &#039;;&#039;)
				comment = true;
				
			// If we&#039;re not in comment mode, add it to our array.
			if (!comment)
				cmdbuffer[serial_count++] = c;
		  }</description>
        <link>https://reprap.org/forum/read.php?12,29397,29397#msg-29397</link>
        <lastBuildDate>Wed, 22 Jul 2026 04:11:14 -0400</lastBuildDate>
        <generator>Phorum 5.2.23</generator>
        <item>
            <guid>https://reprap.org/forum/read.php?12,29397,29458#msg-29458</guid>
            <title>Re: Remarks are  ; surely we should use //  or /* */</title>
            <link>https://reprap.org/forum/read.php?12,29397,29458#msg-29458</link>
            <description><![CDATA[ &gt;No because it is part of the de facto G code standard for machine tool control that has been around since the early sixties.<br />
<br />
Wow I though g codes was a reprap invention. Thanks for the heads up<br />
<br />
Stephen]]></description>
            <dc:creator>stephen george</dc:creator>
            <category>RepRap Host</category>
            <pubDate>Sun, 01 Nov 2009 16:55:11 -0500</pubDate>
        </item>
        <item>
            <guid>https://reprap.org/forum/read.php?12,29397,29433#msg-29433</guid>
            <title>Re: Remarks are  ; surely we should use //  or /* */</title>
            <link>https://reprap.org/forum/read.php?12,29397,29433#msg-29433</link>
            <description><![CDATA[ No because it is part of the de facto G code standard for machine tool control that has been around since the early sixties.]]></description>
            <dc:creator>emt</dc:creator>
            <category>RepRap Host</category>
            <pubDate>Sun, 01 Nov 2009 04:39:50 -0500</pubDate>
        </item>
        <item>
            <guid>https://reprap.org/forum/read.php?12,29397,29422#msg-29422</guid>
            <title>Re: Remarks are  ; surely we should use //  or /* */</title>
            <link>https://reprap.org/forum/read.php?12,29397,29422#msg-29422</link>
            <description><![CDATA[ Ian<br />
<br />
&gt; That refers to a comment in the G code string from the host/ terminal. The code tells the parser to ignore it. <br />
<br />
Yep I see that but shouldn't that be <br />
<br />
G01 X500. F100 // This line moves X 500mm at 100mm a minute<br />
<br />
Rather than<br />
<br />
G01 X500. F100 ; This line moves X 500mm at 100mm a minute<br />
<br />
Stephen]]></description>
            <dc:creator>stephen george</dc:creator>
            <category>RepRap Host</category>
            <pubDate>Sat, 31 Oct 2009 20:40:35 -0400</pubDate>
        </item>
        <item>
            <guid>https://reprap.org/forum/read.php?12,29397,29402#msg-29402</guid>
            <title>Re: Remarks are  ; surely we should use //  or /* */</title>
            <link>https://reprap.org/forum/read.php?12,29397,29402#msg-29402</link>
            <description><![CDATA[ That refers to a comment in the G code string from the host/ terminal. The code tells the parser to ignore it.<br />
<br />
<br />
G01 X500. F100 ; This line moves X 500mm at 100mm a minute]]></description>
            <dc:creator>emt</dc:creator>
            <category>RepRap Host</category>
            <pubDate>Sat, 31 Oct 2009 04:31:12 -0400</pubDate>
        </item>
        <item>
            <guid>https://reprap.org/forum/read.php?12,29397,29397#msg-29397</guid>
            <title>Remarks are  ; surely we should use //  or /* */</title>
            <link>https://reprap.org/forum/read.php?12,29397,29397#msg-29397</link>
            <description><![CDATA[ Guys,<br />
<br />
Going through the firmware code and it looks like the code is using ; as a remark.<br />
Surely we should be following the C convention of // or /* and */.<br />
<br />
; is a line terminator in C and PL/SQL<br />
<br />
regards<br />
<br />
Stephen<br />
<br />
<br />
Here is the code in question in process_string<br />
<br />
<br />
		  //newlines are ends of commands.<br />
		  if (c != '\n')<br />
		  {<br />
			// Start of comment - ignore any bytes received from now on<br />
			if (c == ';')<br />
				comment = true;<br />
				<br />
			// If we're not in comment mode, add it to our array.<br />
			if (!comment)<br />
				cmdbuffer[serial_count++] = c;<br />
		  }]]></description>
            <dc:creator>stephen george</dc:creator>
            <category>RepRap Host</category>
            <pubDate>Sat, 31 Oct 2009 01:56:56 -0400</pubDate>
        </item>
    </channel>
</rss>
