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

Не проверяется сигнатура external функции при вызове #156

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chestozo
Copy link
Contributor

No description provided.

@pasaran
Copy link
Owner

pasaran commented Mar 1, 2013

Не уверен, что это баг.
Подумаю.

@chestozo
Copy link
Contributor Author

chestozo commented Mar 1, 2013

Я тоже рад, что оно работало )
Только тогда непонятно, зачем явно декларировать сигнатуру функции?
Или для параметров с типом nodeset делаются какие-то дополнительные действия?

@pasaran
Copy link
Owner

pasaran commented Mar 1, 2013

Смысл сигнатуры в том, что задать недефолтный (т.е. не scalar) тип
аргументов.
Если ты передаешь больше, чем указано, они просто все приведутся к scalar.
Должна ли быть ошибка?
Ну, пока нет явного способа сказать, что функция принимает неограниченное
число аргументов -- видимо, нет, не нужно ошибки.

2013/3/1 chestozo [email protected]

Я тоже рад, что оно работало )
Только тогда непонятно, зачем явно декларировать сигнатуру функции?
Или для параметров с типом nodeset делаются какие-то дополнительные
действия?


Reply to this email directly or view it on GitHubhttps://github.com//pull/156#issuecomment-14292093
.

@chestozo
Copy link
Contributor Author

chestozo commented Mar 1, 2013

Смысл сигнатуры в том, что задать недефолтный (т.е. не scalar) тип
аргументов.

А зачем?

@pasaran
Copy link
Owner

pasaran commented Mar 1, 2013

Ну потому что у всех выражений нужно на стадии компиляции знать тип.

2013/3/1 chestozo [email protected]

Смысл сигнатуры в том, что задать недефолтный (т.е. не scalar) тип
аргументов.

А зачем?


Reply to this email directly or view it on GitHubhttps://github.com//pull/156#issuecomment-14294348
.

@wayx
Copy link

wayx commented Jun 5, 2013

А почему scalar принимает string/int но не принимает boolean?

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