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

r.terraflow: use default #2660

Merged
merged 1 commit into from
Jan 5, 2023
Merged

Conversation

nilason
Copy link
Contributor

@nilason nilason commented Nov 21, 2022

Force compiler create constructors and assignment operators, fixes following warnings:

  • -Wclass-memaccess
  • -Wdeprecated-declaration

@nilason nilason marked this pull request as ready for review November 21, 2022 10:29
@nilason nilason added enhancement New feature or request C++ Related code is in C++ labels Nov 21, 2022
@nilason nilason added this to the 8.3.0 milestone Nov 21, 2022
@nilason nilason mentioned this pull request Dec 14, 2022
21 tasks
use 'default' for implicitly-declared constructors

fix -Wclass-memaccess warnings

use default (implicit) destructor

simplify; modernise constructs, defaults

format
@nilason
Copy link
Contributor Author

nilason commented Dec 26, 2022

If there are no objections, I’ll merge this shortly.

@nilason nilason merged commit a1e7a1e into OSGeo:main Jan 5, 2023
@nilason nilason deleted the r_terraflow_use_default branch January 5, 2023 15:30
a0x8o added a commit to a0x8o/grass that referenced this pull request Jan 5, 2023
Fixes -Wclass-memaccess and -Wdeprecated-declaration warnings.

Use 'default' for implicitly-declared constructors and use default
(implicit) destructor.

Simplify; modernise constructs.
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Feb 17, 2023
Fixes -Wclass-memaccess and -Wdeprecated-declaration warnings.

Use 'default' for implicitly-declared constructors and use default
(implicit) destructor.

Simplify; modernise constructs.
neteler pushed a commit to nilason/grass that referenced this pull request Nov 7, 2023
Fixes -Wclass-memaccess and -Wdeprecated-declaration warnings.

Use 'default' for implicitly-declared constructors and use default
(implicit) destructor.

Simplify; modernise constructs.
a0x8o added a commit to a0x8o/grass that referenced this pull request May 9, 2024
Fixes -Wclass-memaccess and -Wdeprecated-declaration warnings.

Use 'default' for implicitly-declared constructors and use default
(implicit) destructor.

Simplify; modernise constructs.
a0x8o added a commit to a0x8o/grass that referenced this pull request May 21, 2024
Fixes -Wclass-memaccess and -Wdeprecated-declaration warnings.

Use 'default' for implicitly-declared constructors and use default
(implicit) destructor.

Simplify; modernise constructs.
a0x8o added a commit to a0x8o/grass that referenced this pull request Jun 3, 2024
Fixes -Wclass-memaccess and -Wdeprecated-declaration warnings.

Use 'default' for implicitly-declared constructors and use default
(implicit) destructor.

Simplify; modernise constructs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ Related code is in C++ enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant