Commit Graph

95 Commits

Author SHA1 Message Date
Mael Rouxel-Labbé 8166579e55 Merge branch 'PM-Fix_pmaps_reference_types-GF' into PM-Fix_pmaps_reference_types_master-GF 2021-10-11 19:58:07 +02:00
Mael Rouxel-Labbé 590ddf8015 Various fixes for pmaps:
- Use `value_type` when the returned type is not a reference
- Enforce `reference = value_type` if the returned type is not
  a reference (to avoid `typename PM::reference r = get(m, k)`,
  which will take a ref to a temporary if the `reference`
  typedef is an actualy reference)
- Do not use `put_get_helper` if the property map is not a `lvalue`
  **mutable** property map: the `put()` is `map[k] = v`, which
  is broken if `operator[]` does not return a reference
- The concept `boost::lvalue_property_map_tag` requires `operator[](key)`,
  not a reference in `get(map, key)`. You can have a readable property map
  returning a reference through its `get(map, key)`, but if there is
  no `operator[]`, it's just a `boost::readable_property_map_tag`
- Some const correctness to avoid copying maps with state
  or heavy keys in `get(map, key)` / `put(map, key, value)`
- Base the category of a wrapping pmap on what it offers instead
  of just forwarding the base property map's category
- Tried to do something like mutable lvalue pmap:
  * `value_type& operator[](key&)`
  * `ref get(map, const key&)`
  * `put(map, const key&, const value_type&)`
  and non-mutable lvalue pmap:
  * `const value_type& operator[](const key&)`
  * `ref get(map, const key&)`
  but not everything fits properly...
2021-10-08 15:38:47 +02:00
Laurent Rineau 412881645c Merge pull request #5369 from sgiraudot/Point_set_3-Fix_broken_properties_and_types-GF
[Point Set 3] Fix broken properties and types
2021-02-18 15:23:28 +01:00
Simon Giraudot d257d999fb Fix function properties_and_types() 2021-01-18 15:49:55 +01:00
Mael Rouxel-Labbé 0d66e19361 IO documentation fixes
See @sloriot's review:

https://cgal.geometryfactory.com/CGAL/Members/wiki/Features/Polgyon_soup_mesh_IO/2nd_round/Answer_to_review

(+ some commits before and after this one)
2020-07-24 17:06:00 +02:00
Mael Rouxel-Labbé 26355e2e32 Uniformize capital / third person usage of \brief 2020-06-25 15:13:47 +02:00
Mael Rouxel-Labbé e09eb39a80 Misc cleaning 2020-06-21 11:39:14 +02:00
Mael Rouxel-Labbé 07017616ec Organize Point_set_3 I/O + various fixes along the way 2020-06-19 15:34:49 +02:00
Mael Rouxel-Labbé 038869cab9 Merge remote-tracking branch 'cgal/master' into CGAL-Improve_NP_doc_presentation-GF 2020-05-12 15:12:21 +02:00
Sébastien Loriot 157bdbda08 Merge remote-tracking branch 'cgal/releases/CGAL-4.14-branch' 2020-04-22 13:26:52 +02:00
Simon Giraudot 589eb8c01f Fix warnings with float kernel 2020-04-16 09:23:40 +02:00
Mael Rouxel-Labbé 16092f53d6 Update doc about NPs 2020-04-05 17:56:11 +02:00
Sébastien Loriot 0779373835 extra run of the script to remove tabs and trailing whitespaces
right after the  merge of 4.14 release branch

+ manual fix on one line in:
    * Arrangement_on_surface_2/include/CGAL/IO/Arr_text_formatter.h
    * .travis/generate_travis.sh
2020-03-26 14:16:06 +01:00
Sébastien Loriot 590612a563 run the script to remove tabs and trailing whitespaces 2020-03-26 13:25:16 +01:00
Laurent Rineau 57779dd5c6 Merge pull request #4437 from sgiraudot/Point_set_3-Fix_missing_copy_constructor-GF
Point_set_3: add missing copy constructor
2020-01-15 17:31:52 +01:00
Simon Giraudot e728b74ad6 Point_set_3: add missing copy constructor 2019-12-30 16:42:05 +01:00
Sébastien Loriot 254d60f642 First pass on removing license notice in header for GPL files 2019-10-19 15:23:19 +02:00
Sébastien Loriot 7356421d80 introduce Commercial license SPDX tag 2019-10-19 12:15:19 +02:00
Sébastien Loriot 9bd9c68b83 update LGPL[23]+ and GPL[23]+ SPDX tags
ack -l --no-svg "SPDX-License-Identifier: GPL-3.0\+" | xargs sed -i "s/SPDX-License-Identifier: GPL-3.0+/SPDX-License-Identifier: GPL-3.0-or-later/"
ack -l --no-svg "SPDX-License-Identifier: LGPL-3.0\+" | xargs sed -i "s/SPDX-License-Identifier: LGPL-3.0+/SPDX-License-Identifier: LGPL-3.0-or-later/"
ack -l --no-svg "SPDX-License-Identifier: GPL-2.0\+" | xargs sed -i "s/SPDX-License-Identifier: GPL-2.0+/SPDX-License-Identifier: GPL-2.0-or-later/"
ack -l --no-svg "SPDX-License-Identifier: LGPL-2.0\+" | xargs sed -i "s/SPDX-License-Identifier: LGPL-2.0+/SPDX-License-Identifier: LGPL-2.0-or-later/"
2019-10-18 21:57:54 +02:00
Sébastien Loriot 25a05e9f4c Merge branch 'CGAL-named_function_parameters-4.14' into master
Conflicts resolved by hand
2019-07-28 23:46:43 +02:00
Sébastien Loriot 2f81a21523 Merge 'CGAL-named_function_parameters' for 4.13-branch into 4.14-branch
Conflicts resolved by hand
2019-07-28 22:30:13 +02:00
Sébastien Loriot 560c77f567 rename named function parameter class 2019-07-15 06:25:02 +02:00
Sébastien Loriot 983645a51a match @{ and @} 2019-07-05 21:49:46 +02:00
Andreas Fabri e2d19b0c8d Replace NULL and Nullptr_t with nullptr and nullptr_t 2019-06-04 23:45:44 +02:00
Laurent Rineau 90782d4b7f Revert the merge of two pull-requests that target CGAL-4.14-branch
- Revert "Merge pull request #3857 from MaelRL/CGAL-Fix_duplicate_non_manifold_vertices-GF"
- Revert "Merge pull request #3898 from lrineau/Mesh_3-stop_during_protection-lrineau"
2019-05-14 17:50:45 +02:00
Laurent Rineau f5836b2206 Merge branch 'releases/CGAL-4.13-branch' 2019-03-07 14:31:16 +01:00
Sébastien Loriot 3887a21429 add missing text for brief of advanced class/type/function/concept 2019-03-05 08:01:17 +01:00
Simon Giraudot 458e1c7b80 Update with change of ref tag 2019-02-07 11:36:05 +01:00
Simon Giraudot 8c0417b6c5 Document garbage_size() properly 2019-02-07 11:36:04 +01:00
Simon Giraudot 1b7a2eb2c5 Replace cancel_removal() by cancel_removals() 2019-02-07 11:36:04 +01:00
Simon Giraudot ee2d6eb3ab Add useful typedefs of templates of Point_set_3 2019-02-07 11:36:04 +01:00
Simon Giraudot 776a8b735c Fix doc note about iterator invalidations 2019-02-07 11:36:04 +01:00
Simon Giraudot 6a016e4c27 Update user manual 2019-02-07 11:36:04 +01:00
Simon Giraudot 6f2eed5540 Add method to get properties and types as string 2019-02-07 11:36:04 +01:00
Simon Giraudot f778e5e151 Add methods to easily transfer properties from one point set to another 2019-02-07 11:36:04 +01:00
Simon Giraudot ed748c8ce9 Unify API of add_normal_map() with add_property_map() 2019-02-07 11:34:56 +01:00
Simon Giraudot e11e12b69f Better remove() / is_removed() methods 2019-02-07 11:34:56 +01:00
Sébastien Loriot 2f9408f867 rename link name to match package name 2018-10-08 10:28:55 +02:00
Sébastien Loriot a2e8a1c68f add the suffix Ref to doxygen link to reference 2018-10-08 10:26:56 +02:00
Simon Giraudot effb28cdf2 Fix insertion method, keeping the index 2018-09-06 09:58:49 +02:00
Simon Giraudot c52b32b744 Add trick to get FT right for general PointRange AND for CGAL::Point_set_3 2018-01-11 14:04:19 +01:00
Simon Giraudot 0e81f39e07 Document parameters() in CGAL::Point_set_3 2018-01-10 16:39:32 +01:00
Simon Giraudot fffefbaeaf add all_default() method with point set 3 overload 2018-01-10 16:38:36 +01:00
Laurent Rineau 1c2e9df8cf Merge branch 'releases/CGAL-4.11-branch' 2017-12-19 16:29:10 +01:00
Simon Giraudot 9a7f07cd1d Reinitialize properties of Point_set_3 2017-12-08 10:18:09 +01:00
Sébastien Loriot ee57fc2d6c add SPDX identifier for files under the GPL-3+ license 2017-11-12 10:17:51 +01:00
Simon Giraudot a61d6e02a4 Fix Point_set_3::remove(const Index&) and add test 2017-07-04 15:18:51 +02:00
Simon Giraudot 800c16f2e1 Warning fix: explicit conversion to size_type 2017-04-07 11:49:59 +02:00
Simon Giraudot af78f66230 Use boost::uint32_t instead of std::size_t as size type 2017-04-05 15:29:53 +02:00
Simon Giraudot 4cfbe98564 Point_set_3: fix documentation bug (concept Index accepts constructor with std::size_t) 2017-04-05 15:18:28 +02:00