Repository navigation
Several fixes including: allocate RootedValue for pyTypeFactory call on the stack instead of the heap #234
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
Merged
philippedistributive
merged 143 commits into
caleb/fix/this
from
philippe/pyTypeFactory-cleanup
Apr 3, 2024
Merged
Changes from all commits
Commits
Show all changes
143 commits
Select commit
Hold shift + click to select a range
e4f4d90
added globalThis tests
philippe-distributive a5b12a5
cleanup
philippe-distributive 71269e7
value parameter to PyTypeFactory no longer needs to be allocated on t…
philippe-distributive cb3feef
cleanup
philippe-distributive 13938db
should have been part of the previous commit
philippe-distributive c1fbc1f
pyTypeFactory use HandleValue instead of pointer
philippe-distributive 636fcde
cleanup
philippe-distributive b5bd094
should have bee part of previous commit
philippe-distributive fade8f0
cleanup
philippe-distributive fee7bf3
typo fix
philippe-distributive 695c71a
crash fix
philippe-distributive 234da46
test for crash fix
philippe-distributive 0998b84
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive 839d3e8
cleanup
philippe-distributive a7d7a1a
should have been part of the previous commit
philippe-distributive 7b17b0f
added array test
philippe-distributive d0ed527
cleanup after test
philippe-distributive a7348de
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive 428005a
cleanup
philippe-distributive 8e827f9
formatting
philippe-distributive 15db758
added iterator test
philippe-distributive 050096f
crash fix and its test
philippe-distributive 20a8227
assign to list beyond current length js-style
philippe-distributive 2d934fe
comment
philippe-distributive 6395687
cleanup
philippe-distributive 10c8c79
improved naming
philippe-distributive de32916
formatting
philippe-distributive c827e66
added missing pyExceptions
philippe-distributive 8f2408f
no Py_SET_SIZE on python 3.8
philippe-distributive 95a1b4d
removed incorrect comment
philippe-distributive 1dc7dd1
missing file headers improved
philippe-distributive 2d1462d
type improvement
philippe-distributive 9ff38bc
improved comment
philippe-distributive 18fde47
cleanup: no need for overriding methods that merely call super. prope…
philippe-distributive 4d1d9c2
cleanup
philippe-distributive cc13130
further cleanup
philippe-distributive 32d2d41
added missing file headers. cleanup
philippe-distributive 7843271
further cleanup
philippe-distributive 809b5fa
improved comment
philippe-distributive 976327b
fixed copyright headers
philippe-distributive 1884f79
uniform copyright headers
philippe-distributive bcf55ae
last missing copyrights. license statements fixes
philippe-distributive e2f446e
improved comment
philippe-distributive 5153405
logic improvement
philippe-distributive 454b943
improved naming
philippe-distributive a26680e
cleanup
philippe-distributive 20013a4
project at advanced stage
philippe-distributive d946421
copyright headers missing and documentation cleanup
philippe-distributive 1f42778
further cleanup and missing copyrights
philippe-distributive c484f7e
more missing copyrights
philippe-distributive b30b570
can now assign arguments directly from sys.arg
philippe-distributive a16436d
updates
philippe-distributive a93fcd2
English fix and improved organization
philippe-distributive ea2a2ec
no more leak with the second parameter of pyTypeFactory
philippe-distributive b9250f9
fixed english
philippe-distributive cb57161
more precise error reporting
philippe-distributive e28e160
simplification
philippe-distributive cdceef1
crash fix
philippe-distributive 481f04b
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive 54cb096
cleanup
philippe-distributive 7172cff
jsObject/jsArray should be persistentrooted
philippe-distributive 28fd039
cleanup unused methods
philippe-distributive b0a8e0f
cleanup and minor improvements
philippe-distributive 593da22
dict conversion no longer needed now that we have proxies
philippe-distributive 6952028
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive f787578
improved setdefault performance
philippe-distributive e248b44
decref the result of python calls. Rooted type simplifications
philippe-distributive 943bb1d
fix file header attributions
philippe-distributive 850508c
improved clear methods and related naming, variable folding
philippe-distributive 763bd44
fix on clear method, incref in object.get
philippe-distributive ee5c738
heap-allocated root should be persistent
philippe-distributive 1276a9a
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive 889c912
remove unused second parameter to pyTypeFactory
philippe-distributive 4ba97f7
increment firefox version
philippe-distributive 4df71ee
version update
philippe-distributive ba1afb8
use PersistentRooted pointer instead of inline in struct for Object a…
philippe-distributive 30c120d
tp_name must be same as that of super type
philippe-distributive f8a1217
finished details of JSObjectProxy.repr: skip over the $super key to a…
philippe-distributive 9733902
next version will be 0.4.0
philippe-distributive 1849dbf
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive ec03ef9
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive c4dc2bd
hash not implemented entry not required
philippe-distributive 40bdbb3
test fix
philippe-distributive 0765086
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive 7e7ed33
improved reference management
philippe-distributive baab95f
dec ref needs to check for null
philippe-distributive cb92513
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive 9764d1f
post-merge patch-up
philippe-distributive dec4f72
Proxies now use parent type name for enhanced type compatibility
philippe-distributive a909fc7
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive bfc7798
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive c2d102d
cleanup and post-merge adjust
philippe-distributive 5fa62df
post-merge fixup
philippe-distributive c645fca
include cleanup
philippe-distributive b986a4c
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive 9c0524f
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive 8622867
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive bf16c39
enhanced python stack trace tests
philippe-distributive 7abbd1b
Revert "enhanced python stack trace tests"
philippe-distributive f88cfd2
test fix
philippe-distributive 25f6134
Full JS stacks provided in the context of a Python Exception while ca…
philippe-distributive 7eaab5e
improved comment
philippe-distributive 93fb621
one more comment
philippe-distributive 015eb14
added test for array subclass
philippe-distributive 73f068a
Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-a…
philippedistributive b58b0ac
adjustment to __class__ implementation and added test
philippe-distributive 453ded4
Update src/ExceptionType.cc
philippedistributive 327ab26
proper exception stack type
philippe-distributive fbcb81c
Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-a…
philippedistributive 8dfaa43
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippedistributive 993da0e
Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-a…
philippedistributive b52383c
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippedistributive cee3fb0
Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-a…
philippedistributive 66dbb97
adjustment to __class__ for arrayproxy
philippe-distributive 501f26d
Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-a…
philippedistributive e08f74b
cleanup
philippe-distributive 20a47d4
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippedistributive 8c7bce7
Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-a…
philippedistributive 77f460b
Update src/jsTypeFactory.cc
philippedistributive c627e30
Revert "Update src/jsTypeFactory.cc"
philippe-distributive f9c29ad
remove commented-out code
philippe-distributive 3cd6397
Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-a…
philippedistributive e08a1e7
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippedistributive 76aa711
Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-a…
philippedistributive 073972b
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippedistributive 6d447c6
Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-a…
philippedistributive e5a82e6
remove duplicate JS stack trace
philippe-distributive da5f621
logic improvement
philippe-distributive e8c439a
cleanup
philippe-distributive 0dc70af
logic improvement
philippe-distributive 9d737fd
newline not needed
philippe-distributive 93888de
Merge pull request #296 from Distributive-Network/philippe/full-js-an…
philippedistributive dd5e156
entry newline needed
philippe-distributive 1adaabb
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippedistributive 27019d2
added get method test
philippe-distributive df32a3c
allow object key to shadow method
philippe-distributive 1068741
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippedistributive 78f9938
do distinguish between dict . and [] operators
philippe-distributive c0a1af8
Merge branch 'philippe/pyTypeFactory-cleanup' of github.com:Distribut…
philippe-distributive 3cb7ebc
no need for cpython GC compatibility for these
philippe-distributive 8aab622
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippe-distributive a2d818a
post-merge fixup
philippe-distributive e77be01
Merge branch 'caleb/fix/this' into philippe/pyTypeFactory-cleanup
philippedistributive File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these merely make a super call and so are redundant