Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot select the right procedure overload when using concepts. #14729

Closed
bobeff opened this issue Jun 19, 2020 · 1 comment · Fixed by #24315
Closed

Cannot select the right procedure overload when using concepts. #14729

bobeff opened this issue Jun 19, 2020 · 1 comment · Fixed by #24315
Labels

Comments

@bobeff
Copy link
Contributor

bobeff commented Jun 19, 2020

There is a problem with selecting the right overload of hash and == for a generic objects inferred by a concept.

The not working example is as follows:

import sets, hashes

type
  Iterable[T] = concept x
    for value in items(x):
      type(value) is T

  Foo[T] = object
    t: T

proc myToSet[T](keys: Iterable[T]): HashSet[T] =
  for x in items(keys): result.incl(x)

proc hash[T](foo: Foo[T]): Hash =
  echo "specific hash"

proc `==`[T](lhs, rhs: Foo[T]): bool =
  echo "specific equals"

let
  f = Foo[string](t: "test")
  hs = [f, f].myToSet()
/some_path/nimtest/comilerbug.nim(22, 14) template/generic instantiation of `myToSet` from here
/some_path/nimtest/comilerbug.nim(12, 31) template/generic instantiation of `incl` from here
/some_path/Nim/lib/pure/collections/setimpl.nim(49, 21) template/generic instantiation of `rawGet` from here
/some_path/Nim/lib/pure/collections/hashcommon.nim(73, 14) template/generic instantiation of `genHashImpl` from here
/some_path/Nim/lib/pure/collections/hashcommon.nim(63, 12) Error: type mismatch: got <Foo>
but expected one of: 
proc hash(sBuf: string; sPos, ePos: int): Hash
  first type mismatch at position: 1
  required type for sBuf: string
  but expression 'key' is of type: Foo
proc hash(x: cstring): Hash
  first type mismatch at position: 1
  required type for x: cstring
  but expression 'key' is of type: Foo
proc hash(x: float): Hash
  first type mismatch at position: 1
  required type for x: float
  but expression 'key' is of type: Foo
proc hash(x: pointer): Hash
  first type mismatch at position: 1
  required type for x: pointer
  but expression 'key' is of type: Foo
proc hash(x: string): Hash
  first type mismatch at position: 1
  required type for x: string
  but expression 'key' is of type: Foo
proc hash[A](aBuf: openArray[A]; sPos, ePos: int): Hash
  first type mismatch at position: 1
  required type for aBuf: openArray[A]
  but expression 'key' is of type: Foo
proc hash[A](s: HashSet[A]): Hash
  first type mismatch at position: 1
  required type for s: HashSet[hash.A]
  but expression 'key' is of type: Foo
proc hash[A](s: OrderedSet[A]): Hash
  first type mismatch at position: 1
  required type for s: OrderedSet[hash.A]
  but expression 'key' is of type: Foo
proc hash[A](x: openArray[A]): Hash
  first type mismatch at position: 1
  required type for x: openArray[A]
  but expression 'key' is of type: Foo
proc hash[A](x: set[A]): Hash
  first type mismatch at position: 1
  required type for x: set[A]
  but expression 'key' is of type: Foo
proc hash[T: Ordinal | enum](x: T): Hash
  first type mismatch at position: 1
  required type for x: T: Ordinal or enum
  but expression 'key' is of type: Foo
proc hash[T: proc](x: T): Hash
  first type mismatch at position: 1
  required type for x: T: proc
  but expression 'key' is of type: Foo
proc hash[T: tuple](x: T): Hash
  first type mismatch at position: 1
  required type for x: T: tuple
  but expression 'key' is of type: Foo
proc hash[T](foo: Foo[T]): Hash
  first type mismatch at position: 1
  required type for foo: Foo[hash.T]
  but expression 'key' is of type: Foo

expression: hash(key)

If the type Foo is not generic the example works Ok

import sets, hashes

type
  Iterable[T] = concept x
    for value in items(x):
      type(value) is T

  Foo = object
    t: string

proc myToSet[T](keys: Iterable[T]): HashSet[T] =
  for x in items(keys): result.incl(x)

proc hash(foo: Foo): Hash =
  echo "specific hash"
    
proc `==`(lhs, rhs: Foo): bool =
  echo "specific equals"

let
  f = Foo(t: "test")
  hs = [f, f].myToSet()

producing the output:

specific hash
specific hash
specific equals

The same output is expected for the version when the type Foo is generic.

Alternatively if the Foo is generic but instead using a concept for myToSet parameter, an operArray parameter is used the example also works correct:

import sets, hashes

type
  Foo[T] = object
    t: T

proc myToSet[T](keys: openArray[T]): HashSet[T] =
  for x in items(keys): result.incl(x)

proc hash[T](foo: Foo[T]): Hash =
  echo "specific hash"

proc `==`[T](lhs, rhs: Foo[T]): bool =
  echo "specific equals"

let
  f = Foo[string](t: "test")
  hs = [f, f].myToSet()

specific hash
specific hash
specific equals

This issue is blocking #14481

$ nim -v
Nim Compiler Version 1.3.5 [Linux: amd64]
Compiled at 2020-06-19
Copyright (c) 2006-2020 by Andreas Rumpf

git hash: 99c198625c602984578f9c53b05a28c54de4f4cb
active boot switches: -d:release -d:danger
@bobeff
Copy link
Contributor Author

bobeff commented Jun 19, 2020

I managed to workaround a few similar issues when working on #14481, but despite putting a huge amount of effort I didn't manage to find how to overcome this on a library level. Maybe a fix in the compiler is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants