report warning when using deprecated syntax for property default values

2008-04-13  Juerg Billeter  <j@bitron.ch>

	* vala/valaparser.vala: report warning when using deprecated syntax
	  for property default values

	* vala/valaconstructor.vala, vala/valaenum.vala: port to new syntax

svn path=/trunk/; revision=1207

diff --git a/ChangeLog b/ChangeLog
index 75892868e952c995ba6d4f590d1e80ad3aa6e797..71c75ac339365eb13fe924b9b5e734b87b1c6624 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2008-04-13  Jürg Billeter  <j@bitron.ch>
+
+       * vala/valaparser.vala: report warning when using deprecated syntax
+         for property default values
+
+       * vala/valaconstructor.vala, vala/valaenum.vala: port to new syntax
+
 2008-04-13  Marc-Andre Lureau  <marcandre.lureau@gmail.com>
 
        * vapi/glib-2.0.vapi (Value): add init, register_transform, and

diff --git a/vala/valaconstructor.vala b/vala/valaconstructor.vala
index 1627175eadc970e3fbc0a0a1875bc18da3a73d56..a01e14e1f5c42cdca43cbc0aaf686d158a619810 100644 (file)
--- a/vala/valaconstructor.vala
+++ b/vala/valaconstructor.vala
@@ -39,7 +39,7 @@ public class Vala.Constructor : Symbol {
        /**
         * Specifies whether this is an instance or a class constructor.
         */
-       public bool instance { get; set; default (true); }
+       public bool instance { get; set; default = true; }
        
        /**
         * Creates a new constructor.

diff --git a/vala/valaenum.vala b/vala/valaenum.vala
index 7365c90e49a8e271b5d0b76b45664916b5d19861..0c0e76c02d09047c608b76cb9c405c4bacf955b3 100644 (file)
--- a/vala/valaenum.vala
+++ b/vala/valaenum.vala
@@ -35,7 +35,7 @@ public class Vala.Enum : Typesymbol {
        /**
         * Specifies whether this enum has a registered GType.
         */
-       public bool has_type_id { get; set; default (true); }
+       public bool has_type_id { get; set; default = true; }
 
        private Gee.List<EnumValue> values = new ArrayList<EnumValue> ();
        private Gee.List<Method> methods = new ArrayList<Method> ();

diff --git a/vala/valaparser.vala b/vala/valaparser.vala
index aa84ccc0403e1f4b22ce1bd9e7d2a24c41d1c115..da1c44abc33727491de6a8e87eba5e9068cd8d67 100644 (file)
--- a/vala/valaparser.vala
+++ b/vala/valaparser.vala
@@ -149,6 +149,16 @@ public class Vala.Parser : CodeVisitor {
                return src;
        }
 
+       SourceReference get_current_src () {
+               return new SourceReference (scanner.source_file, tokens[index].begin.line, tokens[index].begin.column, tokens[index].end.line, tokens[index].end.column);
+       }
+
+       SourceReference get_last_src () {
+               int last_index = (index + BUFFER_SIZE - 1) % BUFFER_SIZE;
+
+               return new SourceReference (scanner.source_file, tokens[last_index].begin.line, tokens[last_index].begin.column, tokens[last_index].end.line, tokens[last_index].end.column);
+       }
+
        void rollback (SourceLocation location) {
                while (tokens[index].begin.pos != location.pos) {
                        prev ();
@@ -1199,7 +1209,7 @@ public class Vala.Parser : CodeVisitor {
                        }
                }
                if (!accept (TokenType.CLOSE_BRACE)) {
-                       Report.error (new SourceReference (scanner.source_file, tokens[index].begin.line, tokens[index].begin.column, tokens[index].end.line, tokens[index].end.column), "expected `}'");
+                       Report.error (get_current_src (), "expected `}'");
                }
 
                return block;
@@ -1626,7 +1636,7 @@ public class Vala.Parser : CodeVisitor {
                }
                if (!root) {
                        if (!accept (TokenType.CLOSE_BRACE)) {
-                               Report.error (new SourceReference (scanner.source_file, tokens[index].begin.line, tokens[index].begin.column, tokens[index].end.line, tokens[index].end.column), "expected `}'");
+                               Report.error (get_current_src (), "expected `}'");
                        }
                }
        }
@@ -2054,7 +2064,7 @@ public class Vala.Parser : CodeVisitor {
                                        throw new ParseError.SYNTAX (get_error ("property default value already defined"));
                                }
                                if (accept (TokenType.OPEN_PARENS)) {
-                                       // deprecated
+                                       Report.warning (get_last_src (), "deprecated syntax, use `default = value;`");
                                        prop.default_expression = parse_expression ();
                                        expect (TokenType.CLOSE_PARENS);
                                } else {