[geos-commits] [SCM] GEOS branch main updated. 8781da7eeeaaf66d2147125911c60e089115b505
git at osgeo.org
git at osgeo.org
Tue Feb 18 21:29:28 PST 2025
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GEOS".
The branch, main has been updated
via 8781da7eeeaaf66d2147125911c60e089115b505 (commit)
from 0989e2dbbfef206928bd1849be5118022febc2f7 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 8781da7eeeaaf66d2147125911c60e089115b505
Author: Martin Davis <mtnclimb at gmail.com>
Date: Tue Feb 18 21:28:57 2025 -0800
Update DEVELOPER-NOTES.md
diff --git a/DEVELOPER-NOTES.md b/DEVELOPER-NOTES.md
index a718b815f..4e6d35426 100644
--- a/DEVELOPER-NOTES.md
+++ b/DEVELOPER-NOTES.md
@@ -73,7 +73,8 @@ public:
### Function Parameter calling conventions
* Objects which are passed as read-only with no transfer of ownership
- should be passed as `const` references (`const Geometry&`).
+ should be passed as `const` [references](https://en.wikipedia.org/wiki/Reference_%28C++%29#) (e.g. `const Geometry&`).
+ References are simpler and safer to use than pointers.
* in particular, this applies to Geometry objects passed to API operations
(in static functions, class constructors, or initialization methods)
* NOTE: many GEOS API functions use pass-by-pointer (`Geometry*`), but this is historical baggage
@@ -135,4 +136,4 @@ In **implementation files**, `using` [declarations](https://en.cppreference.com/
```
using geos::algorithm::Distance;
using namespace geos::geom;
-```
\ No newline at end of file
+```
-----------------------------------------------------------------------
Summary of changes:
DEVELOPER-NOTES.md | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
hooks/post-receive
--
GEOS
More information about the geos-commits
mailing list