Skip to content

Fix small NPE on Android#651

Closed
mahesh-hegde wants to merge 3 commits intobeanshell:masterfrom
mahesh-hegde:master
Closed

Fix small NPE on Android#651
mahesh-hegde wants to merge 3 commits intobeanshell:masterfrom
mahesh-hegde:master

Conversation

@mahesh-hegde
Copy link

Fixes a small NPE on Android, which happens because Security.getProperty call returns null there.

@sonatype-lift
Copy link

sonatype-lift bot commented Nov 1, 2022

⚠️ 23 God Classes were detected by Lift in this project. Visit the Lift web console for more details.

@nickl-
Copy link
Member

nickl- commented Dec 1, 2022

Thank you for this!

For some reason the CI didn't run so no way to test the regression, will try get that going.

@nickl-
Copy link
Member

nickl- commented Dec 1, 2022

There appears to be build errors or tests failing. Please advise...

@mahesh-hegde
Copy link
Author

Seems to be concurrency related error. Not sure what's causing it.

I didn't test locally, because some tests fail on my machine.

No problem if this cannot be merged. Just thought someone may want to run it on Android in future.

@nickl-
Copy link
Member

nickl- commented Dec 1, 2022

There may be something wrong with the CI scripts not sure why it stopped running automatically or why it is failing all the open PRs. Will need to make time and come through this...

nickl- added a commit to nickl-/beanshell that referenced this pull request Jan 3, 2023
Fixes beanshell#498 NPE in TargetError with null cause.
Fixes beanshell#322 /*** ***/ is not a formal comment
Fixes beanshell#536 added 5 new operators ??, ??=, ?:, ?., and <=>
Fixes beanshell#609 obfuscate bsh.Primitive to expose math methods
Fixes beanshell#684 can now cast to type Number from primitive
Fixes beanshell#663 identifier defined array dimensions for parameters
Fixes beanshell#651 avoid NPE on package access property and use null
Support for var keyword tested, appears to work with nothing added.
Added Types get primitive type and isNumeric from class
Auto narrowing and widening of primitives cast added
Detect method signature of widening to BigInteger and BigDecimal as assignable and more specific then Object
nickl- added a commit to nickl-/beanshell that referenced this pull request Jan 3, 2023
Fixes beanshell#498 NPE in TargetError with null cause.
Fixes beanshell#322 /*** ***/ is not a formal comment
Fixes beanshell#536 added 5 new operators ??, ??=, ?:, ?., and <=>
Fixes beanshell#609 obfuscate bsh.Primitive to expose math methods
Fixes beanshell#684 can now cast to type Number from primitive
Fixes beanshell#663 identifier defined array dimensions for parameters
Fixes beanshell#651 avoid NPE on package access property and use null
Support for var keyword tested, appears to work with nothing added.
Added Types get primitive type and isNumeric from class
Auto narrowing and widening of primitives cast added
Detect method signature of widening to BigInteger and BigDecimal as assignable and more specific then Object
@nickl- nickl- closed this in #686 Jan 3, 2023
@nickl- nickl- closed this in 67a7c65 Jan 3, 2023
pgiffuni pushed a commit to pgiffuni/beanshell that referenced this pull request Jan 8, 2024
Fixes beanshell#498 NPE in TargetError with null cause.
Fixes beanshell#322 /*** ***/ is not a formal comment
Fixes beanshell#536 added 5 new operators ??, ??=, ?:, ?., and <=>
Fixes beanshell#609 obfuscate bsh.Primitive to expose math methods
Fixes beanshell#684 can now cast to type Number from primitive
Fixes beanshell#663 identifier defined array dimensions for parameters
Fixes beanshell#651 avoid NPE on package access property and use null
Support for var keyword tested, appears to work with nothing added.
Added Types get primitive type and isNumeric from class
Auto narrowing and widening of primitives cast added
Detect method signature of widening to BigInteger and BigDecimal as assignable and more specific then Object
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.

2 participants