Follow-up from "Memory leak and access fixes"
The following discussions from !3932 (merged) should be addressed:
-
@chleh started a discussion: Same here.
-
@chleh started a discussion: (+1 comment) I think I just found similar code in Toms's last MR. Wouldn't it be better to just disallow pointers?
-
@chleh started a discussion: (+2 comments) Here, nullptr could be handled.
-
@chleh started a discussion: Much better than before. But wouldn't a plain object be even better?
-
@chleh started a discussion: Maybe
_raster_data
should be astd::vector
?