Implement modules support and import() function

belliash 코멘트됨,

2018-07-12 19:04:04 +02:00

소유자

In scope of this ticket, we need to implement a module system for P# Interpreter. It is a necessary step in order to divide the existing code into separate, dynamically loadable modules. This includes import() function, that can be called from inside a script. It should return a boolean value (false on any error, or true otherwise). What is more, some 'dummy' module has to be written for tests.

Requirements are:

import() has to check if library is already loaded,
import() should return true on successful library load, or if the module has been already loaded - false otherwise,
import() should take a module name as parameter, then add extension automatically and look for it in some predefined directory
all loaded modules should be kept in some structure (linked list or better a hash map); some mechanisms are already implemented in PH7 Engine,
structure saved in "ph7_vm" has to contain information about: module name, module version, short module description
all these information should be returned by some init() function implemented in module and called from builtin_import()
module should register all its constants and functions
all libraries have to be unloaded on program exit, to free up memory
multiplatform (Unix .so & Windows .dll) support

Some code snippet:

<?php
    var_dump(import('dummy')); // true, module loaded
    var_dump(import('dummy2')); // false - no such module
    var_dump(import('dummy')); // true - module already loaded
?>

In scope of this ticket, we need to implement a module system for P# Interpreter. It is a necessary step in order to divide the existing code into separate, dynamically loadable modules. This includes import() function, that can be called from inside a script. It should return a boolean value (false on any error, or true otherwise). What is more, some 'dummy' module has to be written for tests. Requirements are: * import() has to check if library is already loaded, * import() should return true on successful library load, or if the module has been already loaded - false otherwise, * import() should take a module name as parameter, then add extension automatically and look for it in some predefined directory * all loaded modules should be kept in some structure (linked list or better a hash map); some mechanisms are already implemented in PH7 Engine, * structure saved in "ph7_vm" has to contain information about: module name, module version, short module description * all these information should be returned by some init() function implemented in module and called from builtin_import() * module should register all its constants and functions * all libraries have to be unloaded on program exit, to free up memory * multiplatform (Unix .so & Windows .dll) support Some code snippet: <?php var_dump(import('dummy')); // true, module loaded var_dump(import('dummy2')); // false - no such module var_dump(import('dummy')); // true - module already loaded ?>

belliash added the

enhancement

label 2018-07-12 19:04:04 +02:00

belliash 자체적으로 할당됨 2018-07-15 00:16:12 +02:00

belliash 커밋

2018-07-18 22:02:13 +02:00

에서 이 이슈 언급

Initial support for modules, partially implements #1

belliash 코멘트됨,

2018-07-18 22:09:09 +02:00

Author

소유자

<?php
    var_dump(dummy_function());
    var_dump(import('dummy'));
    var_dump(dummy_function());
?>

Above code gives the following output now:

null
bool(TRUE)
string(30 'Hello world from dummy module!')

<?php var_dump(dummy_function()); var_dump(import('dummy')); var_dump(dummy_function()); ?> Above code gives the following output now: null bool(TRUE) string(30 'Hello world from dummy module!')

belliash 코멘트됨,

2018-07-18 22:10:20 +02:00

Author

소유자

Still TODO:

Unload all modules on exit,
Add Windows support.

Still TODO: * Unload all modules on exit, * Add Windows support.

belliash 코멘트됨,

2018-07-18 22:11:59 +02:00

Author

소유자

Still TODO:

Unload all modules on exit,
Add Windows support,
Check if module is already loaded.

Still TODO: * Unload all modules on exit, * Add Windows support, * Check if module is already loaded.

belliash 코멘트됨,

2018-07-19 17:30:16 +02:00

Author

소유자

I think it is fully implemented now by 28dbeeb1ad & 0dc9a04f0e.

I think it is fully implemented now by 28dbeeb1ad & 0dc9a04f0e.

belliash closed this issue

2018-07-19 17:30:16 +02:00

devnexen 코멘트됨,

2018-07-19 17:33:26 +02:00

멤버

If we re going into WIN32 api vs unixes. Would it be useful to make wrappers around some few types (e.g. HINSTANCE / FILE * or void *) at least ?

If we re going into WIN32 api vs unixes. Would it be useful to make wrappers around some few types (e.g. `HINSTANCE` / `FILE *` or `void *`) at least ?

belliash 코멘트됨,

2018-07-19 17:35:54 +02:00

Author

소유자

Maybe it is not perfect, but PH7 itself is already multiplatform and I think we want to go for it.

devnexen 코멘트됨,

2018-07-19 17:38:13 +02:00

멤버

No the code is good but if we go further and need to call more and more specific win32 api in the future and as HINSTANCE, for example, is quite common ... Or another approach (if it happens not necessary at all for now), to have specific c with all win32 api implementation and one separated for unixes Makefile caring only about the latter the vs project for former ... but might be early to think that way, just food for future thoughts ... sort of.

belliash 코멘트됨,

2018-07-19 17:42:14 +02:00

Author

소유자

I hope there won't be much more WinAPI in use.

👍 1

belliash 자체적으로 할당됨 2018-07-28 19:22:25 +02:00

devnexen 다음으로부터 할당됨 belliash

2018-07-28 19:22:26 +02:00

Implement modules support and import() function #1