Skip to content

Feature/tree 1c#49

Open
fishca wants to merge 5 commits into
e8tools:developfrom
fishca:feature/tree_1c
Open

Feature/tree 1c#49
fishca wants to merge 5 commits into
e8tools:developfrom
fishca:feature/tree_1c

Conversation

@fishca

@fishca fishca commented Nov 9, 2017

Copy link
Copy Markdown

Исправляю свой недосмотр

@fishca

fishca commented Nov 9, 2017

Copy link
Copy Markdown
Author

вот жеж std::max/min на винде и на линухах это совсем не одно и то же :(

@dmpas

dmpas commented Nov 9, 2017

Copy link
Copy Markdown
Collaborator

https://travis-ci.org/dmpas/v8unpack/builds/299566458

таки не собирается с std::max, потому в своё время остался на MAX.

@fishca

fishca commented Nov 9, 2017

Copy link
Copy Markdown
Author

надо поколдовать на эту тему...

@dmpas

dmpas commented Nov 15, 2017

Copy link
Copy Markdown
Collaborator

@fishca как успехи? ты спрашивай, если что. В тул1сд на те же костыли натыкались.

@ava57r

ava57r commented Dec 2, 2017

Copy link
Copy Markdown
Member

std::min and std::max шаблонные. Оба параметра метода должны быть одного типа.
Для шаблонной функции неявное приведение типа не работает.
в тул1цд сделано так

curlen = std::min(static_cast<uint64_t>(DEFAULT_PAGE_SIZE - curoffblock), _length);

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.

3 participants