Skip to content

Commit

Permalink
Do not mark abstract class contstuctors public
Browse files Browse the repository at this point in the history
Starting with Vala 0.45.1, it is an error to mark abstract class
constructors public[0,1]. This commit removes three such
declarations.

fixes #609

[0] https://gitlab.gnome.org/GNOME/vala/raw/master/NEWS
[1] https://gitlab.gnome.org/GNOME/vala/issues/766

Signed-off-by: Randy Barlow <randy@electronsweatshop.com>
  • Loading branch information
bowlofeggs authored and fiaxh committed Sep 1, 2019
1 parent 016ab2c commit 48c2e7a
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion libdino/src/service/content_item_store.vala
Expand Up @@ -207,7 +207,7 @@ public abstract class ContentItem : Object {
public Encryption? encryption { get; set; default=null; }
public Entities.Message.Marked? mark { get; set; default=null; }

public ContentItem(int id, string ty, Jid jid, DateTime sort_time, DateTime display_time, Encryption encryption, Entities.Message.Marked mark) {
ContentItem(int id, string ty, Jid jid, DateTime sort_time, DateTime display_time, Encryption encryption, Entities.Message.Marked mark) {
this.id = id;
this.type_ = ty;
this.jid = jid;
Expand Down
4 changes: 2 additions & 2 deletions qlite/src/column.vala
Expand Up @@ -55,7 +55,7 @@ public abstract class Column<T> {
return res;
}

public Column(string name, int type) {
Column(string name, int type) {
this.name = name;
this.sqlite_type = type;
}
Expand Down Expand Up @@ -185,4 +185,4 @@ public abstract class Column<T> {
}
}

}
}
2 changes: 1 addition & 1 deletion qlite/src/statement_builder.vala
Expand Up @@ -15,7 +15,7 @@ public abstract class StatementBuilder {
public T value;
public Column<T>? column;

public AbstractField(T value) {
AbstractField(T value) {
this.value = value;
}

Expand Down

0 comments on commit 48c2e7a

Please sign in to comment.