Thread overview
website update
Aug 10, 2015
Tony
Aug 10, 2015
tcak
Aug 10, 2015
bachmeier
August 10, 2015
It looks like this page:

http://dlang.org/hash-map.html

Should have the override keyword added the the member functions in Foo:

class Foo
{
    int a, b;

    size_t toHash() { return a + b; }

    bool opEquals(Object o)
    {
        Foo foo = cast(Foo) o;
        return foo && a == foo.a && b == foo.b;
    }
}
August 10, 2015
On Monday, 10 August 2015 at 10:34:56 UTC, Tony wrote:
> It looks like this page:
>
> http://dlang.org/hash-map.html
>
> Should have the override keyword added the the member functions in Foo:
>
> class Foo
> {
>     int a, b;
>
>     size_t toHash() { return a + b; }
>
>     bool opEquals(Object o)
>     {
>         Foo foo = cast(Foo) o;
>         return foo && a == foo.a && b == foo.b;
>     }
> }

Also that return line is confusing. Some parenthesis could be put there.
August 10, 2015
On Monday, 10 August 2015 at 11:20:32 UTC, tcak wrote:
> On Monday, 10 August 2015 at 10:34:56 UTC, Tony wrote:
>> It looks like this page:
>>
>> http://dlang.org/hash-map.html
>>
>> Should have the override keyword added the the member functions in Foo:
>>
>> class Foo
>> {
>>     int a, b;
>>
>>     size_t toHash() { return a + b; }
>>
>>     bool opEquals(Object o)
>>     {
>>         Foo foo = cast(Foo) o;
>>         return foo && a == foo.a && b == foo.b;
>>     }
>> }
>
> Also that return line is confusing. Some parenthesis could be put there.

Click the "Improve this page" button in the upper right corner.