[Tinycc-devel] Debian's tcc patches.

[Rob Landley]

Ok, I'm looking at "http://packages.debian.org/unstable/devel/tcc.html"; and 
specifically 
"http://ftp.debian.org/debian/pool/main/t/tcc/tcc_0.9.23-4.diff.gz"; 
to see if there's anything I should include 
in "http://landley.net/hg/tinycc";.

Changes:

1) The example directory's paths change from /usr/local/bin to /usr/bin.
2) In the Makefile,  install: stops calling libinstall.
3) Adds a debian subdir with random files in it.
4) The __FreeBSD__ tests have || __FreeBSD_kernel__ added to them.
5) The VT_BOOL tweaks.
6) A new hunk:

@@ -8403,6 +8408,9 @@
         no_oblock = 1;
         if ((first && tok != TOK_LSTR && tok != TOK_STR) ||
             tok == '{') {
+            if (tok != '{')
+                error("character array initializer must be a literal,"
+                    " optionally enclosed in braces");
             skip('{');
             no_oblock = 0;
         }

I understand changes #1, #3, and #5.  I already installed #5 to my repository 
months ago, and could go either way on #1.  #3 is something that doesn't 
belong outside of debian.

I don't understand #2 and #6 don't really care enough about #4 to ask (if I 
start caring about MacOS X I'll do so directly).  So my questions are:

Change #2: Why shouldn't install call libinstall?

Change #6: What exactly is this trying to accomplish?  You generate an error 
message if the token isn't {, but the error message itself says that the 
braces are optional?  Could you give me an example of what this fixes?

Thanks,

Rob
-- 
"Perfection is reached, not when there is no longer anything to add, but
when there is no longer anything to take away." - Antoine de Saint-Exupery