<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/usr.bin/make/str.c, branch releng/5.3</title>
<subtitle>FreeBSD source tree</subtitle>
<id>https://cgit-dev.freebsd.org/src/atom?h=releng%2F5.3</id>
<link rel='self' href='https://cgit-dev.freebsd.org/src/atom?h=releng%2F5.3'/>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/'/>
<updated>2004-01-25T12:28:36Z</updated>
<entry>
<title>Fixed a bug that previous revision has introduced -- we missed</title>
<updated>2004-01-25T12:28:36Z</updated>
<author>
<name>Ruslan Ermilov</name>
<email>ru@FreeBSD.org</email>
</author>
<published>2004-01-25T12:28:36Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=a59f8cc52c3425f8195320c9ec30418f00e59001'/>
<id>urn:sha1:a59f8cc52c3425f8195320c9ec30418f00e59001</id>
<content type='text'>
the double quotes ("" and '') as a separate argument.

Reported by:	ache

The fix in this and previous revisions combined is functionally
equivalent to the below patch against rev. 1.27 but the code is
now much easier to follow:

%%%
Index: str.c
===================================================================
RCS file: /home/ncvs/src/usr.bin/make/str.c,v
retrieving revision 1.27
diff -u -r1.27 str.c
--- str.c	28 Oct 2002 23:33:57 -0000	1.27
+++ str.c	25 Jan 2004 12:09:21 -0000
@@ -168,7 +168,7 @@
 				inquote = (char) ch;
 				/* Don't miss "" or '' */
 				if (start == NULL &amp;&amp; p[1] == inquote) {
-					start = t + 1;
+					start = t;
 					break;
 				}
 			}
%%%
</content>
</entry>
<entry>
<title>Fixed broken logic when parsing double quotes.</title>
<updated>2004-01-22T18:17:44Z</updated>
<author>
<name>Ruslan Ermilov</name>
<email>ru@FreeBSD.org</email>
</author>
<published>2004-01-22T18:17:44Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=cc7282659a9cb522b3bf71da8ee8c696e8b1882a'/>
<id>urn:sha1:cc7282659a9cb522b3bf71da8ee8c696e8b1882a</id>
<content type='text'>
PR:		bin/61673
MFC after:	1 month
</content>
</entry>
<entry>
<title>Split var.c into var.c and var_modify.c and move all the modification funcs</title>
<updated>2002-10-28T23:33:57Z</updated>
<author>
<name>Juli Mallett</name>
<email>jmallett@FreeBSD.org</email>
</author>
<published>2002-10-28T23:33:57Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=85779f0eadefc3602cfa1c44e1af36d854b79356'/>
<id>urn:sha1:85779f0eadefc3602cfa1c44e1af36d854b79356</id>
<content type='text'>
to var_modify.c, for readability.  constify some low hanging fruit (string
manipulation functions) and the upper layers appropriately.  No longer use
the private strstr(3) implementation, while changing string code.

Tested by:      lots of successful make buildworld.
</content>
</entry>
<entry>
<title>Remove efree(), it isn't used consistently enough to even pretend that it</title>
<updated>2002-10-23T23:16:43Z</updated>
<author>
<name>Juli Mallett</name>
<email>jmallett@FreeBSD.org</email>
</author>
<published>2002-10-23T23:16:43Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=cbfcb39874d97238abccb006b6acf2b2f78b2e65'/>
<id>urn:sha1:cbfcb39874d97238abccb006b6acf2b2f78b2e65</id>
<content type='text'>
might help on the systems it could possibly be used as a bandaid for.  In
fact, the only thing it's useful for is instrumenting free(3) calls, and in
that capacity, it's better served as a local patch, than a public wrapper.
</content>
</entry>
<entry>
<title>Convert make(1) to use ANSI style function declarations.  Variable</title>
<updated>2002-10-09T03:42:10Z</updated>
<author>
<name>Juli Mallett</name>
<email>jmallett@FreeBSD.org</email>
</author>
<published>2002-10-09T03:42:10Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=4526ed6ffb47e16844e942555ad0fab102401734'/>
<id>urn:sha1:4526ed6ffb47e16844e942555ad0fab102401734</id>
<content type='text'>
documentation already adequatedly existed in the description in most
cases.  Where it did not, it was added.  If no documentation existed
beforehand, then none was added.  Some unused dummies for use in the
traversal functions were marked as __unused during the conversion.
Occasionally, local style fixes were applied to lines already being
modified or influenced.

Now make(1) should always build with WARNS=3.
</content>
</entry>
<entry>
<title>Add empty default cases where they should be, remove non-local execution stuff</title>
<updated>2002-09-28T20:03:26Z</updated>
<author>
<name>Juli Mallett</name>
<email>jmallett@FreeBSD.org</email>
</author>
<published>2002-09-28T20:03:26Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=1e3d8881cfbe2762be6f3fa4cdc0a1b153a5b512'/>
<id>urn:sha1:1e3d8881cfbe2762be6f3fa4cdc0a1b153a5b512</id>
<content type='text'>
in compat.c which doesn't even have preprocessor-conditional-hidden support
code, and add a debugging statement where we might end up with a nil list
somehow, but where I doubt it.

First confirmed userland kill for Flexelint.

Sponsored by:	Bright Path Solutions
</content>
</entry>
<entry>
<title>str_concat() doesn't really take const arguments.</title>
<updated>2002-05-22T15:34:00Z</updated>
<author>
<name>Juli Mallett</name>
<email>jmallett@FreeBSD.org</email>
</author>
<published>2002-05-22T15:34:00Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=8eccf3b24874a2ecbc71927ca6cff9d8cd7dbfba'/>
<id>urn:sha1:8eccf3b24874a2ecbc71927ca6cff9d8cd7dbfba</id>
<content type='text'>
Submitted by:	bde
Pointy hat to:	jmallett
</content>
</entry>
<entry>
<title>Replace the evil that is __DECONST() with (void *).  This is one of the least</title>
<updated>2002-05-22T11:16:48Z</updated>
<author>
<name>Juli Mallett</name>
<email>jmallett@FreeBSD.org</email>
</author>
<published>2002-05-22T11:16:48Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=27ffa61f2a909c165d049a83e377be8b6155af45'/>
<id>urn:sha1:27ffa61f2a909c165d049a83e377be8b6155af45</id>
<content type='text'>
evil things we can do involving the const qualifier and a pointer.

Submitted by:	bde, ru
</content>
</entry>
<entry>
<title>Fix make(1) behaviour regarding SysV-style substitution when given a nil</title>
<updated>2002-05-05T19:04:58Z</updated>
<author>
<name>Juli Mallett</name>
<email>jmallett@FreeBSD.org</email>
</author>
<published>2002-05-05T19:04:58Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=417485c74a78cd91a5ea222ed3f85bb540128202'/>
<id>urn:sha1:417485c74a78cd91a5ea222ed3f85bb540128202</id>
<content type='text'>
left-hand-side.

PR:		bin/5297
Submitted by:	"Matthew Emmerton" &lt;matt@gsicomp.on.ca&gt;
MFC after:	1 week
</content>
</entry>
<entry>
<title>Back out rev 1.17, it breaks dependencies.</title>
<updated>2002-04-14T04:15:34Z</updated>
<author>
<name>David E. O'Brien</name>
<email>obrien@FreeBSD.org</email>
</author>
<published>2002-04-14T04:15:34Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=75a778e9f82112f4f490b964bad7f2ccc9f90c4e'/>
<id>urn:sha1:75a778e9f82112f4f490b964bad7f2ccc9f90c4e</id>
<content type='text'>
With rev 1.7 one cannot build src/bin/sh -- because make fails to create
the buildtools before trying to use them.  Actually it does compile the
buildtools into .o's before trying to use them, but not all the way into
binaries.
</content>
</entry>
</feed>
