objectionary / eo

EOLANG, an Experimental Pure Object-Oriented Programming Language Based on 𝜑-calculus
https://www.eolang.org
MIT License
1.01k stars 127 forks source link

Implemented necessary unix system calls #3306

Closed c71n93 closed 3 months ago

c71n93 commented 3 months ago

Related to #3236

Adds infrastructure for system calls implementation. Also presents some basic unix system calls: getpid, read, write.


PR-Codex overview

This PR updates system calls and dependencies in EO runtime.

Detailed summary

The following files were skipped due to too many changes: eo-runtime/src/main/java/EOorg/EOeolang/EOsys/DispatchedUnixSyscall.java, eo-runtime/src/test/java/EOorg/EOeolang/EOsys/EOosTest.java, eo-runtime/src/main/java/EOorg/EOeolang/EOsys/CStdLib.java, eo-runtime/src/main/java/EOorg/EOeolang/EOsys/EOposix.java, eo-runtime/src/test/java/org/eolang/SnippetTestCase.java, eo-runtime/src/test/java/EOorg/EOeolang/EOsys/EOposixTest.java, eo-runtime/src/test/java/EOorg/EOeolang/EOsys/DispatchedUnixSyscallTest.java, eo-runtime/src/main/java/EOorg/EOeolang/EOsys/DispatchedNativeDefault.java

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

c71n93 commented 3 months ago

@maxonfjvipon could you check this one, please?

maxonfjvipon commented 3 months ago

@c71n93 please merge last version of master branch. Some break changes happened

c71n93 commented 3 months ago

@maxonfjvipon all resolved, please check

c71n93 commented 3 months ago

@yegor256 could you check this one, please?

c71n93 commented 3 months ago

@yegor256 could you merge this one please? It is necessary to continue moving windows system calls into this repo (#3236)

yegor256 commented 3 months ago

@c71n93 probably a typo in file name, should be posix-test.eo

c71n93 commented 3 months ago

@yegor256 thank you. all fixed, check please.

yegor256 commented 3 months ago

@rultor merge

rultor commented 3 months ago

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 3 months ago

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 31min)

0crat commented 3 months ago

@maxonfjvipon Great job on the review! 🎉 You've earned 63 points: +25 base, +18 for reviewing 930 hits-of-code, and +20 for your 25 insightful comments. We didn't deduct points for self-review or low comment count, which shows your thoroughness. Your balance is now +175. Keep up the excellent work!

0crat commented 3 months ago

@c71n93 Great contribution! You've earned +4 points: +25 base, -7 for 970 hits-of-code (≥200), -15 for 970 hits-of-code (≥800), -8 for 33 review comments, +10 minimum bonus. Remember, quality over quantity helps maximize rewards. Your balance: +101. Keep up the good work! 💪