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

Refactor method calls to fix MQL5 deprecated behavior warnings #69

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

barmenteros
Copy link
Contributor

This commit refactors hidden method calls in HashMap.mqh to use explicit base class method invocation, addressing MQL5 compiler warnings about deprecated behavior that will be disallowed in future versions. The specific warnings being resolved are for hidden method calls that do not occur when compiling with MQL4, thus maintaining backward compatibility with MQL4 while ensuring future-proof compatibility with MQL5.

The changes are limited to method call semantics and do not alter any functional aspects of the code. By making these updates, the codebase stays compliant with both MQL4 and future MQL5 compiler standards.

This commit refactors hidden method calls in `HashMap.mqh` to use explicit base class method invocation, addressing MQL5 compiler warnings about deprecated behavior that will be disallowed in future versions. The specific warnings being resolved are for hidden method calls that do not occur when compiling with MQL4, thus maintaining backward compatibility with MQL4 while ensuring future-proof compatibility with MQL5.

The changes are limited to method call semantics and do not alter any functional aspects of the code. By making these updates, the codebase stays compliant with both MQL4 and future MQL5 compiler standards.
@dingmaotu dingmaotu merged commit ec696fa into dingmaotu:master Apr 18, 2024
@dingmaotu
Copy link
Owner

Thanks for your contribution.

@barmenteros
Copy link
Contributor Author

You're welcome. We appreciate the timely merge of our pull request.

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