Thread (1 message) 1 message, 1 author, 2011-09-09

Re: [PATCH v3 3/3] dtc: Support character literals in bytestrings

From: David Gibson <hidden>
Date: 2011-09-09 07:01:18

On Thu, Sep 08, 2011 at 11:37:21PM -0700, Anton Staaf wrote:
On Thu, Sep 8, 2011 at 6:15 PM, David Gibson
[off-list ref] wrote:
quoted
On Thu, Sep 08, 2011 at 02:39:17PM -0700, Anton Staaf wrote:
quoted
With this patch the following property assignment:

    property = ['a' 2b '\r'];

is equivalent to:

    property = [61 2b 0d];
[snip]
quoted
+<BYTESTRING>{CHAR_LITERAL} {
You should just be able to make the existing rule a <*> one,
rather than having a separate rule for BYTESTRING context.
I thought about that too, but I was concerned that it would then allow
character literals in properties outside of the cell list or
bytestring syntax.  Which I suppose is exactly what you had suggested
before now that I think about it.  And would end up being no more
ambiguous, and possibly more internally consistent.  I'll make the
change and add a test case that shows that functionality as well if
that makes sense to you.
Well, it would allow character literals to be *lexed* in other places,
but the parser would still reject them outside of the (so far)
intended contexts without further changes.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help