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