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

assert(false) in implemented methods #1514

Merged
merged 1 commit into from
Jun 8, 2016
Merged

Conversation

aka-demik
Copy link
Contributor

Win32TCPConnection.tcpNoDelay and Win32TCPConnection.keepAlive look implemented. But ends with line assert (false).
In current state, http_server example fails, when using win32 configuration.

`Win32TCPConnection.tcpNoDelay` and `Win32TCPConnection.keepAlive` look implemented. But ends with line `assert (false)`.
In current state, [http_server](https://github.com/rejectedsoftware/vibe.d/tree/master/examples/http_server) example fails, when using `win32` configuration.
@aka-demik
Copy link
Contributor Author

Trace:

Task terminated with unhandled exception: Assertion failure

core.exception.AssertError@C:\Users\Denis.Hlyakin\AppData\Roaming\dub\packages\vibe-d-0.7.28\vibe-d\source\vibe\core\drivers\win32.d(1162): Assertion failure
----------------
0x004EE193 in _d_assert
0x0047F98D in @property void vibe.core.drivers.win32.Win32TCPConnection.tcpNoDelay(bool) at C:\Users\Denis.Hlyakin\AppData\Roaming\dub\packages\vibe-d-0.7.28\vibe-d\source\vibe\core\drivers\win32.d(1163)
0x00429FD2 in void vibe.http.server.handleHTTPConnection(vibe.core.net.TCPConnection, vibe.http.server.HTTPListenInfo) at C:\Users\Denis.Hlyakin\AppData\Roaming\dub\packages\vibe-d-0.7.28\vibe-d\source\vibe\http\server.d(1521)
0x004239C9 in void vibe.http.server.listenHTTPPlain(vibe.http.server.HTTPServerSettings).doListen(vibe.http.server.HTTPListenInfo, bool, bool).__lambda4(vibe.core.net.TCPConnection) at C:\Users\Denis.Hlyakin\AppData\Roaming\dub\packages\vibe-d-0.7.28\vibe-d\source\vibe\http\server.d(1425)
0x0048134E in void vibe.core.drivers.win32.Win32TCPConnection.runConnectionCallback(vibe.core.net.TCPListenOptions) at C:\Users\Denis.Hlyakin\AppData\Roaming\dub\packages\vibe-d-0.7.28\vibe-d\source\vibe\core\drivers\win32.d(1478)
0x00495470 in void vibe.core.core.makeTaskFuncInfo!(void delegate(vibe.core.net.TCPListenOptions), vibe.core.net.TCPListenOptions).makeTaskFuncInfo(ref void delegate(vibe.core.net.TCPListenOptions), ref vibe.core.net.TCPListenOptions).callDelegate(vibe.core.core.TaskFuncInfo*) at C:\Users\Denis.Hlyakin\AppData\Roaming\dub\packages\vibe-d-0.7.28\vibe-d\source\vibe\core\core.d(499)
0x0044EC8E in void vibe.core.core.CoreTask.run() at C:\Users\Denis.Hlyakin\AppData\Roaming\dub\packages\vibe-d-0.7.28\vibe-d\source\vibe\core\core.d(1085)
0x004F3429 in void core.thread.Fiber.run()

@aka-demik
Copy link
Contributor Author

aka-demik commented Jun 8, 2016

Win32EventDriver.listenUDP() and Win32FileStream.seek() use assert(false). Body of Win32UDPConnection.onUDPRead is empty. But this does not affect http_server example.
As I understand it, these methods are not implemented.

@s-ludwig
Copy link
Member

s-ludwig commented Jun 8, 2016

Thanks, I couldn't find any explanation for these apart from being accidentally left over.

@s-ludwig s-ludwig merged commit ac54371 into vibe-d:master Jun 8, 2016
@aka-demik aka-demik deleted the patch-1 branch June 8, 2016 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants