Code causes errors in FDT due to inappropriate use of 'final' keyword as method variable #109

Closed
darronschall opened this Issue Jul 28, 2010 · 4 comments

Comments

Projects
None yet
1 participant
@darronschall
Contributor

darronschall commented Jul 28, 2010

Originally filed by alec...@googlemail.com on 2009-07-24T07:38:14

In URI.as, lines 1906-8 a method variable is defined as:

var final:String = this.scheme + ":" + ref;
return constructURI(final);

When the project is imported into FDT this is flagged as an error because 'final' is supposed to be a
reserved keyword, and should not be used as a method variable.

@darronschall

This comment has been minimized.

Show comment Hide comment
@darronschall

darronschall Jul 28, 2010

Contributor

Updated by alec...@googlemail.com on 2009-07-24T07:41:56

proposed patch for trivial change

Contributor

darronschall commented Jul 28, 2010

Updated by alec...@googlemail.com on 2009-07-24T07:41:56

proposed patch for trivial change

@darronschall

This comment has been minimized.

Show comment Hide comment
@darronschall

darronschall Jul 28, 2010

Contributor

Updated by mikechambers on 2009-09-21T04:45:53

Original ticket set owner to mikechambers
Original ticket set status to Accepted (we converted to open)

Contributor

darronschall commented Jul 28, 2010

Updated by mikechambers on 2009-09-21T04:45:53

Original ticket set owner to mikechambers
Original ticket set status to Accepted (we converted to open)

@darronschall

This comment has been minimized.

Show comment Hide comment
@darronschall

darronschall Jul 28, 2010

Contributor

Updated by mikechambers on 2009-09-22T07:45:27

This issue was closed by revision r111.

Original ticket set status to Fixed (we converted to closed)

Contributor

darronschall commented Jul 28, 2010

Updated by mikechambers on 2009-09-22T07:45:27

This issue was closed by revision r111.

Original ticket set status to Fixed (we converted to closed)

@darronschall

This comment has been minimized.

Show comment Hide comment
@darronschall

darronschall Jul 28, 2010

Contributor

Updated by mikechambers on 2009-09-22T07:45:48

Fixed. Committed revision 111.

Thanks for the report and patch.

Contributor

darronschall commented Jul 28, 2010

Updated by mikechambers on 2009-09-22T07:45:48

Fixed. Committed revision 111.

Thanks for the report and patch.

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment