Return-Path: X-Original-To: apmail-incubator-flex-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-flex-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id D45AFE76D for ; Sat, 1 Dec 2012 03:06:19 +0000 (UTC) Received: (qmail 24949 invoked by uid 500); 1 Dec 2012 03:06:19 -0000 Delivered-To: apmail-incubator-flex-dev-archive@incubator.apache.org Received: (qmail 24828 invoked by uid 500); 1 Dec 2012 03:06:18 -0000 Mailing-List: contact flex-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: flex-dev@incubator.apache.org Delivered-To: mailing list flex-dev@incubator.apache.org Received: (qmail 24762 invoked by uid 99); 1 Dec 2012 03:06:16 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 01 Dec 2012 03:06:16 +0000 X-ASF-Spam-Status: No, hits=0.3 required=5.0 tests=FRT_ADOBE2,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of cyrill.zadra@gmail.com designates 209.85.212.47 as permitted sender) Received: from [209.85.212.47] (HELO mail-vb0-f47.google.com) (209.85.212.47) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 01 Dec 2012 03:06:08 +0000 Received: by mail-vb0-f47.google.com with SMTP id e21so117208vbm.6 for ; Fri, 30 Nov 2012 19:05:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=TFCdr4Aaj7Xf4bq8Wh0jxnIFeC2purNhiRj8/hbqrq8=; b=dUELdSNkpPhqFgNDNrYswZ8mcWICoolVXx66mj1qwEJxAkHHu5T8M6r7MgDrobObk8 TwAJXTFKBzEuM6nuN5S4EA0tsyXVPDLiZgwdRBgpRl/N6020zi3LmNFrT6gDE2kPNdO1 GaXRtvGeTTJnJlEIVyLLXAQHKWgzBw6/tE9/ghPd7TTZOL2Ke4Usf8SCS4gqYXTRosmz jNTq4JksvK7il0cjrpVzkEKE1PoHitmXKPsXXoJnyW5R2v5KjgcRKWMZxbi7oyxl8MKh 0idI9jHLy24pqMIJqLUNqLvCVaxeGMDdybBmWvSiM6FdCCCACW5cvJZu/T/YRzns46vA XZTQ== MIME-Version: 1.0 Received: by 10.52.70.83 with SMTP id k19mr2393976vdu.89.1354331146987; Fri, 30 Nov 2012 19:05:46 -0800 (PST) Received: by 10.220.140.208 with HTTP; Fri, 30 Nov 2012 19:05:46 -0800 (PST) In-Reply-To: <149F8129B58B2D418508E63117D9C5419B5B4CFD91@nambx05.corp.adobe.com> References: <149F8129B58B2D418508E63117D9C5419B5B4CFCC6@nambx05.corp.adobe.com> <149F8129B58B2D418508E63117D9C5419B5B4CFD89@nambx05.corp.adobe.com> <149F8129B58B2D418508E63117D9C5419B5B4CFD91@nambx05.corp.adobe.com> Date: Fri, 30 Nov 2012 19:05:46 -0800 Message-ID: Subject: Re: Falcon test failure From: Cyrill Zadra To: "flex-dev@incubator.apache.org" Content-Type: text/plain; charset=ISO-8859-1 X-Virus-Checked: Checked by ClamAV on apache.org Yes the old compiler allows both examples... so I'll commit my change for CSSFunctionCallPropertyValue and then this last failing test will also will turn to "Ok" :). On Fri, Nov 30, 2012 at 5:37 PM, Gordon Smith wrote: > I.e., > > > Button { fontFamily: 'Times Roman' } > > > instead of > > > Button { fontFamily: "Times Roman" } > > > ? If the old compiler allowed this, Falcon should probably allow it. > > - Gordon > > -----Original Message----- > From: Gordon Smith [mailto:gosmith@adobe.com] > Sent: Friday, November 30, 2012 5:18 PM > To: flex-dev@incubator.apache.org > Subject: RE: Falcon test failure > > I'm not an expert on CSS syntax. Are you supposed to be able to use either double quotes or single quotes for style values? > > - Gordon > > -----Original Message----- > From: Cyrill Zadra [mailto:cyrill.zadra@gmail.com] > Sent: Friday, November 30, 2012 4:12 PM > To: flex-dev@incubator.apache.org > Subject: Re: Falcon test failure > > Hi Gordon > > Oh I forgot to comment the one line.... > > I exended the method getSingleArgumentFromRaw() to remove single quotes. > But before I commit that I first wanted to ask first here on the list if this method is really supposed to do that. I'm probably too cautious in commiting ;-) > > So single quotes should they also be removed if they exists at the beginning and end of a rawArgument? > > Cyrill > > > > Am 30.11.2012 um 13:03 schrieb Gordon Smith : > >> @Cyril: Thanks for adding some unit tests for CSS nodes. One test, CSSFunctionCallPropertyValue_static_getSingleArgumentFromRaw(), is failing for me. It's expecting "Test" but getting "'Test'". >> >> Does it pass for you? >> >> - Gordon