Normalize prototype before decomposing as a function

[odersky]

Fixes #13526

[odersky]

Looks like the requests CB project now fails for all new PRs. What could be the cause of this?

[odersky]

Here's the error:

[error] -- [E069] Syntax Error: /Users/odersky/workspace/dotty/library/src/scala/IArray.scala:13:7 
[error] 13 |object IArray:
[error]    |       ^
[error]    |object IArray cannot have the same name as object IArray in package scala -- cannot define object member with the same name as a object member in self reference _.
[error]    |(Note: this can be resolved by using another name)
[error] -- [E069] Syntax Error: /Users/odersky/workspace/dotty/library/src/scala/IArray.scala:13:7 
[error] 13 |object IArray:
[error]    |       ^
[error]    |object IArray cannot have the same name as object IArray in package scala -- cannot define object member with the same name as a object member in self reference _.
[error]    |(Note: this can be resolved by using another name)
[warn] three warnings found

This does very much look like a mis-configured test. Should we disable requests for now?

[griggt]

Did the compilation of requests actually fail with that error? I just see a bunch of timeouts during the tests which looks as if the httpbin.org service was offline (the requests test suite seems to depend on this external service)

https://github.com/lampepfl/dotty/runs/3769019088#step:8:2614

 X requests.RequestTests.matchingMethodWorks 25670ms 
  requests.TimeoutException: Request to http://httpbin.org/put timed out. (readTimeout: 10000, connectTimout: 10000)

[odersky]

@griggt The IArray problem was what I saw when I ran it locally. So probably that's because I ran it with a non-empty classpath. Good to know what the real reason was, and hopefully it gets fixed soon.

[griggt]

I restarted the CI workflow, it's green now.