https://issues.dlang.org/show_bug.cgi?id=18757

RazvanN <razvan.nitu1305@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |razvan.nitu1305@gmail.com

--- Comment #1 from RazvanN <razvan.nitu1305@gmail.com> ---
(In reply to FeepingCreature from comment #0)
> Right now, you can define a struct to have a static invariant:
> 
> struct S {
>   static invariant { assert(false); }
>   static void foo() { }
> }
> 
> But the invariant will not be checked when calling foo.
> 
> Either this should work, or "static invariant {}" should be a syntax error.

Placing static in front of an invariant has no effect on the invariant. The problem with the above code is that the method foo is marked as static:

struct S
{
    invariant { (assert (false); }
    static void foo() {}
}

void main()
{
    S s;
    s.foo(); \\ invariant does not get called
}

while :

struct S
{
    static invariant { (assert (false); }
    void foo() {}
}

void main()
{
    S s;
    s.foo(); \\ invariant does get called
}

So we can conclude that invariants are not called for static functions. The spec does not mention anything about this so there are 2 possibilities :

1. Invariants were designed only for struct/class instances (in which case the bug report will be closed with a spec update)

2. Invariants should also be called when static methods are called (in which case this should be fixed)

--

https://issues.dlang.org/show_bug.cgi?id=18757

--- Comment #2 from FeepingCreature <default_357-line@yahoo.de> ---
In case 1, I maintain that this is a bug - the compiler should not smile-and-nod while silently ignoring syntax that hints at a semantic that does not apply.

--

https://issues.dlang.org/show_bug.cgi?id=18757

uplink.coder@googlemail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |uplink.coder@googlemail.com

--- Comment #3 from uplink.coder@googlemail.com ---
Well invariants cannot apply to static methods since they are only concerned
about the object's state.
which by definition cannot be directly touched in static functions.
that's why it's not called.

I don't see that there is a basis to issue a warning.

--

https://issues.dlang.org/show_bug.cgi?id=18757

--- Comment #4 from RazvanN <razvan.nitu1305@gmail.com> ---
(In reply to uplink.coder from comment #3)
> Well invariants cannot apply to static methods since they are only concerned
> about the object's state.
> which by definition cannot be directly touched in static functions.
> that's why it's not called.
> 
> I don't see that there is a basis to issue a warning.

What about object that have static fields? If you have a static function which modify static fields? I would argue that an invariant might want to check whether a static function has modified the state of the object.

--

https://issues.dlang.org/show_bug.cgi?id=18757

Jonathan M Davis <issues.dlang@jmdavisProg.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |issues.dlang@jmdavisProg.co
                   |                            |m

--- Comment #5 from Jonathan M Davis <issues.dlang@jmdavisProg.com> ---
(In reply to FeepingCreature from comment #2)
> In case 1, I maintain that this is a bug - the compiler should not smile-and-nod while silently ignoring syntax that hints at a semantic that does not apply.

The reality of the matter is that the compiler ignores attributes like statict all over the place (e.g. static on most things at module scope is allowed and ignored, and you can do something like put @safe on a member variable without complaint from the compiler). If anything, it's extremely typical for the compiler to ignore attributes that don't apply rather than to treat them as an error. In some ways, this can be annoying, but it does help avoid problems in generic code. So, whether an error should be given in this particular case and not others is up for debate, but it's par for the course around here.

(In reply to RazvanN from comment #4)
> What about object that have static fields? If you have a static function which modify static fields? I would argue that an invariant might want to check whether a static function has modified the state of the object.

We could certainly add a feature like that, but it's certainly not what invariant was designed for. Its purpose was specifically to verify the state of an object, and arguably, the static fields of an object aren't any different from those at module-sope except for the fact that they're scoped to be inside a struct or class. And as such, having an invariant for them would be pretty weird and is of pretty debatable value. If we went down that route, the question would become why we don't have module-level invariants.

--

https://issues.dlang.org/show_bug.cgi?id=18757

--- Comment #6 from FeepingCreature <default_357-line@yahoo.de> ---
In my opinion, static this sets the precedent that static + class-level feature = module-level feature.

--

https://issues.dlang.org/show_bug.cgi?id=18757

Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bugzilla@digitalmars.com
           Hardware|x86_64                      |All
                 OS|Linux                       |All

--- Comment #7 from Walter Bright <bugzilla@digitalmars.com> ---
A `static invariant` does not have special semantics distinct from `invariant` other than it does not have a `this` parameter. That fully explains the observed behavior. The `static` is not ignored:

  struct S {
    int i;
    static invariant { assert (i == 3); } // Error: need this for i of type int
    void foo() {}
  }

  void main() {
    S s;
    s.foo(); // invariant does get called
  }

Whether that is desirable behavior or not is another matter entirely. Our general practice when confronted with these sorts of issues is to make it an error until we figure out what is the best approach, if there even is one, rather than inventing arbitrary behavior without much of any supporting rationale or use cases.

--

https://issues.dlang.org/show_bug.cgi?id=18757

Andrei Alexandrescu <andrei@erdani.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andrei@erdani.com

--- Comment #8 from Andrei Alexandrescu <andrei@erdani.com> ---
The problem is that static methods don't call the static invariant.

--

https://issues.dlang.org/show_bug.cgi?id=18757

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P1                          |P3

--

https://issues.dlang.org/show_bug.cgi?id=18757

--- Comment #9 from dlangBugzillaToGithub <robert.schadek@posteo.de> ---
THIS ISSUE HAS BEEN MOVED TO GITHUB

https://github.com/dlang/dmd/issues/17856

DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB

--