changelog.docu

/** \page om_changelog Changelog

\htmlonly

<table border=1 cellpadding=4 cellspacing=2>

<!-- --------------------------------------------------------------------- -->


<tr valign=top><td><b>6.0</b> (?/?/?)</td><td>

<b>Core</b>
<ul>
<li>Added a couple of methods to enable more efficient adding of batches of vertices (Without resizing property vectors) </li>
</ul>

<b>IO</b>
<ul>
<li>Obj reader: added texCoord3d functions to objloader</li>
</ul>

<b>VectorT</b>
<ul>
<li>Full C++11 rewrite of VectorT (Keeping C++98 backward compatibility)</li>
<li>Split VectorT implementation into two files (One c++98 and one c++11)</li>
<li>Added += operator to mesh iterators (C++11 only) </li>
</ul>

<b>Unittests</b>
<ul>
<li>Added several unittests for VectorT</li>
<li>Added Benchmarks for VectorT</li>
</ul>

<b>Build System</b>
<ul>
<li>Updated c++11 ifdefs to use c++11 (starting with VS1015)</li>
</ul>

<b>Python Interface</b>
<ul>
<li>Fixed segfault in decimater due to a bad pointer</li>
</ul>

</tr>


<tr valign=top><td><b>5.2</b> (?/?/?)</td><td>

<b>Core</b>
<ul>
<li>Fixed ugly typo in __decrement function of Basehandle. Seems to be unused in the rest of the code.</li>
<li>drop VectorDimensionsT metaprogram in favor of vector_traits in normal computation function</li>
</ul>


<b>IO</b>
<ul>
<li>PLY Reader: Fixed CR LF newline handling for binary file headers.</li>
<li>Obj writer: Fix OBJ material files and face color (Thanks to Sven-Kristofer Pilz for the patch) </li>
</ul>

<b>Tools</b>
<ul>
<li>Decimater: Added missing preprocess collapse call in size_t DecimaterT<Mesh>::decimate(size_t _n_collapses)  (Thanks to Andrej Ivanis for the hint)</li>
</ul>



<b>Build System</b>
<ul>
<li>Fixed Typo in cmake install for smoother (Thanks to Takashi Michikawa for the fix).</li>
</ul>

<b>Documentation</b>
<ul>
<li>Fixed order of smoother parameters in the documentation. (Thanks to Takashi Michikawa for the info).</li>
</ul>

</tr>


<tr valign=top><td><b>5.1</b> (2015/17/12)</td><td>

<b>Core</b>
<ul>
<li>Added clean_keep_reservation() (remove all elements from the mesh but keeps the properties. In contrast to clean() the memory used for the elements will remain allocated</li>
<li>Reduced number of include statements to speedup compilation</li>
<li>Removed unused member variable next_cache_count_ member of PolyConnectivity and made it local</li>
</ul>

<b>General</b>
<ul>
<li>Fixed bug preventing from building with 2 dimensional vectors (e.g. vec2i) </li>
<li>Heap: Minor cleanup (some consts)</li>
<li>Added static versions of ArrayKernel::edge_handle and ...::halfedge_handle</li>
<li>Reduced number of includes</li>
</ul>

<b>IO</b>
<ul>
<li>OMWriter: fix crash in OMWriter when writing an empty mesh with colors</li>
</ul>

<b>VectorT</b>
<ul>
<li>take VectorT argument by reference for non-member vector scalar multiplication</li>
</ul>

<b>Unittests</b>
<ul>
<li>Added some basic linear algebra tests for VectorT</li>
<li>Added tests for vec2x construction</li>
</ul>

<b>Documentation</b>
<ul>
<li>Add some more documentation about primitive deletion. Closes #14</li>
</ul>

<b>Build System</b>
<ul>
<li>Fixed missing debug dll for autobuilders</li>
</ul>

<b>Python Interface</b>
<ul>
<li>Exposed Decimater to python</li>
</ul>

</tr>



<tr valign=top><td><b>5.0</b> (2015/11/18)</td><td>

<b>General</b>
<ul>
<li>Moved repository to git</li>
</ul>

<b>Core</b>
<ul>
<li>Fixed bug where range iterators contained an invalid element when the range was empty</li>
<li>Reintroduced and polished the StatusSet API (Thanks to Martin Marinov)</li>
<li>More efficient implementation of opposite_halfedge_handle (Less instructions required)</li>
<li>PolyMeshes: More robust algorithm for computing surface normals (Fixes bad normals on concave polygons)</li>
</ul>

<b>VectorT</b>
<ul>
<li>Modernized VectorT constructors for C++11 (Less redundancy, make some of the VectorT constructors constexpr)</li>
<li>C++11: Added _htmlColor literal operator.</li>
<li>C++11: Added explicit static_cast along with static SFINAE checks to VectorT constructor.</li>
</ul>

<b>IO</b>
<ul>
<li>loaded properties are presistant per default</li>
<li>remove unnecessary c headers</li>
<li>PLY Writer: fix crash when saving a mesh and a custom property was removed before, extend unittest for this case</li>
<li>PLY Writer: add custom property (with pod data type) support for binary ply files closes #2496</li>
<li>remove redundant code refs #2525</li>
<li>Off Writer: Removed redundant code</li>
<li>Made STLReader, OBJReader reuse stringstreams to improve performance</li>
</ul>

<b>Unittests</b>
<ul>
<li>add obj writer unittest</li>
<li>add unittest for write stla</li>
<li>Fixed bug where Some C++11 unittests were not build and used</li>
<li>Added more unittests c++11 ranges vvrange and vertexrange</li>
</ul>


</tr>


<tr valign=top><td><b>4.1</b> (2015/07/27,Rev.1318)</td><td>
<b>Core</b>
<ul>
<li>Fixed bug in polymesh normal computation. Added more unittests for normal computation.</li>
<li>Get rid of some windows preprocessor warnings</li>
</ul>

<b>IO</b>
<ul>
<li>Options: Fixed bad enum</li>
</ul>


<b>Build System</b>
<ul>
<li>Removed fixbundle calls if not building apps.</li>
</ul>

</tr>


<tr valign=top><td><b>4.0</b> (2015/07/08,Rev.1308)</td><td>

<b>LICENSE</b>
<ul>
<li>OpenMesh has been relicensed to BSD-3</li>
</ul>

<b>Core</b>
<ul>
<li>Fixed crash when writing with multiple threads to mostream. closes #2293 (Note: This fix is only active with C++11 support) </li>
<li>fix "relink failed" condition closes #1244. This patch fixes a rare condition in add_face, where a non-manifold configuration could be created. </li>
<li>Added CW and CCW Circulators ( These new iterator versions also fix the problems that circulators could get valid again, if iterating past the end,
    they are also faster, as the require one check less then the old ones.)</li>
<li>Implement a warning on the operator-- of the old iterators. It was possible to use them to iterate from begin into negative direction. At boundaries,
    that did not work correctly and visited one entity twice. The warning can be disabled via cmake or define (-DNO_DECREMENT_DEPRECATED_WARNINGS ).</li>
<li>Fixed wrong normal calculation in calc_face_normal for Poly Meshes. closes #2427</li>
<li>Fixed various warnings with vector casts</li>
<li>General: Get rid of some compiler warnings</li>
<li>VectorT: fix compile error on vector scalar multiplication when scalar type differs of vector scalar type and the scalar was lhs. closes #2367</li>
<li>Removed dead and uncompilable code regarding the status set. refs #2511 </li>
</ul>

<b>IO</b>
<ul>
<li>Readers/Writers: extend "can_read/can_write" function. returns true, if no explicit filename is given, but just the extension. closes #2431 </li>
<li>Writers: Avoid std::endl in various writers to avoid flushing the output stream on every line. (Thanks to Roman Zoller for the hint).</li>
<li>OBJReader: fix reading mtl files with texture only for obj files</li>
<li>OBJReader: fix crash, if obj file contains degenerated faces</li>
<li>OBJReader: improve performance for small vectors/faces</li>
<li>OBJReader: load vertex color values as float in range of [0..1] instead of integers [0..255]</li>
<li>PLY Reader: Ascii reader now supports custom attributes</li>
<li>PLY Writer: add custom property support for ascii version</li>
<li>PLY Reader: unify int8_t/signed char/char in ply reader and writer. </li>
<li>PLY Reader/Writer Binary: change type int8_t to signed char from type char closes #2499 </li>
<li>STL Writer: Add a space after the solid keyword to make some readers happy. </li>
</ul>

<b>Utils</b>
<ul>
<li>Fixing PropertyManager so that it cooperates with bool properties even on unforgiving implementations (such as clang).</li>
</ul>


<b>Tools</b>
<ul>
<li>Don't use auto_ptr when compiler supports C++11. Replaced by unique_ptr.</li>
<li>Fix for Smoother set_relative error (Thanks to Leonardo Cecchinato for the fix) </li>
<li>Improved SmootherT.hh Documentation</li>
<li>Fixed old OM3 port warning in smoother</li>
<li>Decimater: ModProgMesh - Fixed write on 64-bit systems</li>
</ul>

<b>General</b>
<ul>
<li>Fixed various build warnings</li>
<li>Moved several iostream includes to cc files or removed them where not necessary (or switched to omlog and omerr)</li>
</ul>

<b>Documentation</b>
<ul>
<li>Example for locking vertices in decimater</li>
<li>Added extra hint about status flags when collapsing edges</li>
<li>Updated tutorial on reading and writing meshes with python bindings</li>
<li>Updated the tutorial with further information on how to build the python bindings</li>
<li>Added missing requestsfor polyMesh in collapsee docu</li>
<li>Add feature support list for readers/writers</li>
<li>add info about requesting standard properties and loading a mesh with options - refs #2512</li>
<li>add info about texture information in OBJ file format - closes #2369</li>
<li>add info about texture index property refs #2512</li>
</ul>

<b>Unittests</b>
<ul>
<li>fixed a bug in the unit test test_add_face.py</li>
<li>Added tests for tutorials about flip and collapse</li>
<li>Added basic smoother compile test</li>
<li>Added unit test for bool PropertyManager::set_range.</li>
<li>Added write unittest for binary stl files</li>
<li>Added c++11 test file</li>
<li>Use the acg macro to add the unittests to make sure flags are passed through (e.g. to enable c++11)</li>
</ul>

<b>Build System</b>
<ul>
<li>Python Bindings: strip pythonlibs version string of possible other characters than 0-9 and .  (e.g. '+')</li>
<li>Python Bindings: don't require boost-python</li>
<li>Python Bindings: updated the cmake file so that cmake finds boost python installations with the names python2 and python3</li>
<li>Python Bindings: fixed a cmake warning on os x (Policy CMP0042: MACOSX_RPATH)</li>
<li>fix windows warning about unknown preprocessor define</li>
<li>Default to release build type if not selected via cmake. Only on non Win platforms</li>
<li>No copy after build required for doc, as its copied by doxygen anyway</li>
<li>Got rid of the old ACGMakefiles, as they were not maintained anymore</li>
</ul>


</tr>

<tr valign=top><td><b>3.3</b> (2015/01/16,Rev.1204)</td><td>

<b>Core</b>
<ul>
<li>change name of the types "True" and "False" to "TrueType" and "FalseType" to clarify, that these are types and avoid conflicts</li>
<li>Fixed two missing this pointers</li>
<li>Removed several unused variables</li>
<li>PolyConnectivity: Fixed constness of EntityRange begin() and end() methods.</li>
<li>VectorT: add specializations for 5/6 dimension vectors</li>
<li>VectorT: added const getter for data_vector()</li>
<li>VectorT: <b> Modified Vector constructors!</b> The constructors now check the size of the vector at compile time not run time. This might produce compiler errors but ensures that the vectors are initialized correctly.</li>
</ul>

<b>Python Interface</b>
<ul>
<li>Added a Python interface for OpenMesh. Implemented via boost python</li>
</ul>

<b>Utils</b>
<ul>
<li>PropertyManager: Added copy_to method.</li>
<li>PropertyManager: Added static copy(...) convenience method.</li>
</ul>

<b>IO</b>
<ul>
<li>OBJ writer: remove excluding condition for face color (Thanks to Javier Granado for the hint). closes #2247 </li>
<li>PLY Reader: add custom property support for PLY reader ASCII version</li>
</ul>

<b>Tools</b>
<ul>
<li>VDPM: fix comparison with invalid iterator closes #2263 </li>
<li>Decimater: ModProgMeshT fix 32/64-bit bug on windows (analyzer and synthesizer reads with unsigned int -> 32bit, writer writes size_t -> possible 64-bit)</li>
<li>Decimater: Added an observer class which can trigger a callback to monitor and abort the decimater process from outside.</li>
</ul>

<b>Documentation</b>
<ul>
<li>New acg logo for OpenMesh</li>
<li>Fixed compilation documentation to include Qt5 and latest compilers</li>
<li>Added smootherdocu. closes #2185</li>
<li>Added documentation about direction of the circulators</li>
<li>Fixed Typo in Mesh Kernel Concept Documentation (return value)</li>
</ul>

<b>Unittests</b>
<ul>
<li>add unittest for vdpm (read and write/read) closes #2263</li>
</ul>

<b>Build system</b>
<ul>
<li>Apple/Unix: Don't install into libs/OpenMesh subdir but directly into libs. closes #2257</li>
<li>Apple/Unix: Install to lib64 on 64-bit. closes #2257</li>
<li>fix wrong name for static lib on unix based os with multi generator compilers</li>
<li>fix permissions for static lib (they have the same as all other libs now)</li>
<li>fix cmake (>=3.0) warning on MacOS idue to missing rpath. closes #2251</li>
<li>Fixed Typo in gtest finder (Thanks to Roman Zoller for the patch)</li>
</ul>

</tr>


<tr valign=top><td><b>3.2</b> (2014/07/18,Rev.1107)</td><td>

<b>Core</b>
<ul>
<li>Fixed copy paste typos in the vertex face and face face range code</li>
</ul>

<b>IO</b>
<ul>
<li>Silence the readers/writers (Some of them always wrote output to command line).</li>
</ul>

<b>Utils</b>
<ul>
<li>PropertyManager: Added PropertyManager::move to C++11 .</li>
</ul>


<b>Subdivider</b>
<ul>
<li>Added CatmullClark without point updates support.</li>
<li>Uniform, Longest Edge: fixed template deduction error with float precision meshes</li>
</ul>

<b>Documentation</b>
<ul>
<li>Updated doxyfile to 1.8.7</li>
<li>Added some documentation to get the to and from vertices to the mesh navigation page</li>
<li>Added boundary navigation info to mesh navigation page</li>
</ul>

<b>Build system</b>
<ul>
<li>Qt5 support for example apps</li>
<li>Improved OpenMesh cmake gtest finder for Xcode 5.1.1 with gtest 1.7</li>
<li>Don't copy qt4 anymore, if build_apps is disabled, but the qmake variable still defined</li>
<li>Add dependencies and fixbundle for apps only if we build apps</li>
</ul>

<b>Unittests</b>
<ul>
<li>Added unittest for textured meshes</li>
<li>Read/Write OBJ: Fixed unsigned signed comparison</li>
</ul>

</tr>

<tr valign=top><td><b>3.1</b> (2014/05/09,Rev.1050)</td><td>

<b>Core</b>
<ul>
<li>Fix for c++11 call to pointer to member function of type 'VertexIter ()' drops 'const' qualifier (Thanks to Ronald Richter and Kenshi Takayama for the fix)</li>
<li>PolyConnectivity: Added range based circulators courtesy of Kenshi Takayama. (Thanks!)</li>
<li>Rearranged the code a bit in order to have all range based stuff in one place.</li>
<li>Fixed calc_normals_... functions (Missing this pointer) Thanks to Martin Schmidt for the report</li>
</ul>

<b>Decimater</b>
<ul>
<li>Fixed Destructor of ModNormaldeviation: Requested normals on destruction</li>
<li>Decimater doesn't need normals anymore, if no module requires normals</li>
</ul>


<b>Utils</b>
<ul>
<li>PropertyManager: Added set_range method</li>
<li>PropertyManager: Added duplicate method</li>
</ul>

<b>Subdivider</b>
<ul>
<li>Added missing include of Property.hh to uniform loop subdivider. (Thanks to Jonathan Balzer for the hint)</li>
</ul>

<b>IO</b>
<ul>
<li>OBJ Writer: Fixed material writer using wrong stream (Thanks to  Vladimir Chalupecky for the fix). closes 2037</li> 
<li>VTK Writer: Added simple VTK writer writing only vertices and faces (Thanks to Vladimir Chalupecky for the code). closes 2039</li>
<li>PLY Reader: Fix locking up when loading some binary files ( Thanks to  Peter Newman for the patch). closes 2141</li>
</ul>


<b>Unittests</b>
<ul>
<li>Added unittests for the normal calculation functions </li>
</ul>

<b>Build system</b>
<ul>
<li>Added OpenMesh finder. Closes 959</li>
</ul>

</tr>

<tr valign=top><td><b>3.0</b> (2014/02/05,Rev.990)</td><td>

<b>Core</b>
<ul>
<li>Rewrote all circulators (STL compliant, removed redundant code)</li>
<li>Added stream operator for FVIter.</li>
<li>Added mesh cast for meshes with different but identical traits.</li>
<li>Fixed segfault of copy_all_properties if one property has been removed before. (Thanks to Simon Flöry for the patch)</li>
<li>Added more convenient versions of calc_edge_vector to PolyMeshT.</li>
<li>Added HalfedgeLoop iterator to iterate over loops of halfedges (faces, hoes)</li>
<li>OpenMesh: Added convenience method for face centroid computation.</li>
</ul>

<b>IO</b>
<ul>
<li>close stl files with endsolid</li>
<li>fix om writer writing colors</li>
<li>Fixed regression and Problem in OM writer</li>
<li>Fixed bug in OM Writer when bool property is written which contains 0x20. (Thanks to Simon Flöry for the patch)</li>
<li>The OpenMesh STLReader now also reads face normals if requested</li>
</ul>

<b>Utils</b>
<ul>
<li>PropertyManager: Added ability to get property name</li>
<li>PropertyManager: Added getMesh property to PropertyManager.</li>
<li>Improved C++11-branch of PropertyManager and at the same time fixed compile error with gcc 4.7</li>
<li>Put mesh_cast methods into surrounding namespace so clang finds them.</li>
<li>Build fix for getopt.h on Apple. (Thanks to Adam Vandenberg for the fix)</li>
</ul>

<b>Documentation</b>
<ul>
<li>Adapt to new iterators</li>
<li>Updated the OpenMesh tutorials to be OM 3 compliant</li>
<li>Use short names in Doxygen to prevent windows build failure due to excessive name length</li>
<li>Updated doxygen files</li>
<li>Use different DOT_FONTNAME</li>
<li>Added hint that collapse needs status attrributes on the mesh</li>
<li>Raised Documentation to Doxyfile 1.8.5</li>
</ul>

<b>Unittests</b>
<ul>
<li>Added more om reader/writer unittests</li>
<li>Mesh Dual test</li>
<li>Stripifier test</li>
<li>Split unittests into seperate compile units</li>
<li>Added mesh_cast unit test</li>
<li>STL normal reader unit tests</li>
</ul>

<b>Build system</b>
<ul>
<li>Build unittests with variadic max of 10 on VS 2012</li>
<li>enable deprecated warning for msvc</li>
<li>Added flag to activate largeaddressaware option on WIN</li>
<li>Removed obsolete ftemplate-depth, as the defaults should be fine</li>
<li>Removed outdated VS2008 project file</li>
<li>Use cmake generator targetnames for copying files</li>
</ul>

</tr>

<tr valign=top><td><b>2.4</b> (2013/08/07,Rev.882)</td><td>

<b>Significant interface changes</b>
<ul>
<li>The functions n_vertices(),n_edges().n_faces() return size_t now.</li>
<li>reserve and resize of the property vectors now take size_t</li>
<li>Fixed various other size_t conversions</li>
</ul>

<b>Vector Type</b>
<ul>
<li>vector_type min(const vector_type& _rhs) and vector_type max(const vector_type& _rhs) are declared const now. (Thanks to Vladimir Chalupecky for the hint)</li>
<li>minimize and maximize return vector_type& (reference) instead of vector_type (value) to allow chaining p.minimize(p1).minimize(p2). (Thanks to Vladimir Chalupecky for the hint)</li>
</ul>

<b>Core</b>
<ul>
<li>Allow PolyConnectivity::delete_edge to mark an edge as deleted, if there are no faces incident.</li>
<li>Don't use c headers in c++ files anymore</li>
</ul>


<b>IO</b>
<ul>
<li>Try to get rid of memory leak in IOManager(Changes the pointer used for IOManager to a static IOManager in the getter function)</li>
<li>Fixed writing face indices in different configurations regarding vertex texture coordinates and vertex normals (Thanks to Robert Luo for the patch)</li>
<li>Fixed a bug with OBJReader that prevented the material color to be loaded correctlyi(Thanks to Karthik Nathan for the patch)</li>
<li>Made STL Reader work, with the clear = false flag and Redundant lookup removed. ( Thanks to Peter Newman for the patch)</li>
<li>Missing include, preventing build on VS2012 (Thanks to Mageri Filali Maltouf for the patch)</li>
<li>Fixed various warnings reported by cppcheck</li>
<li>read_mesh now throws a compile error if an Options::Flag enum is passed as an argument instead of an Options object</li>
<li>Make OpenMesh PLY writer work as expected (Thanks to Chin-chia Wu for the patch)</li>
<li>Added colori and colorAi functions to BaseExporter which return Vec3i and Vec4i respectively</li>
<li>Adjusted the PLYWriter to use colori and colorAi for writing ascii files</li>
<li>Added a ColorFloat flag to Options, which can be set to write and read RGBA values as float instead of unsigned char</li>
<li>PLY writer and reader can now also handle color floats</li>
<li>OFF support for floats. Note that for reading binary OFF files with color floats, the user has to set the flag, that floats are expected</li>
</ul>


<b>Utils</b>
<ul>
<li>PropertyManager: Added PropertyManager::propertyExists() method.</li>
<li>PropertyManager: Enabled initialization of invalid PropertyManager.</li>
<li>Added color_cast from Vec3f and Vec4f to Vec3i and Vec3ui</li>
<li>Added color_cast from Vec4f to Vec4i and Vec4ui</li>

</ul>

<b>Decimater</b>
<ul>
<li>Make Hausdorff module thread safe. Removed static point vector. (Thanks to Falko Löffler for the fix)</li>
</ul>

<b>Tools</b>
<ul>
<li>Command Line Decimater: Added an explanation on how to use multiple modules to the commandlineDecimater help output</li>
<li>Command Line Decimater: The normal deviation module now also is a priority module in the commandlineDecimater tool</li>
<li>Gui Decimater: Added decimater related help output to DecimaterGui when 'h' is pressed as the application is running</li>
</ul>

<b>Unittests</b>
<ul>
<li>Added unittest for skipping iterators</li>
<li>Added unittest for collapse and is_collapse_ok</li>
<li>Added unittests for delete_face</li>
<li>Added unittests for circulators</li>
<li>Added test for writing and reading vertex colors to and from an OFF file. The default color type Vec3uc from DefaultTraits in Traits.hh is used.</li>
<li>Added a ply file written with MeshLab and a corresponding unittest</li>
<li>Added a unittest that writes and reads a binary PLY file with vertex colors</li>
</ul>

<b>Documentation</b>
<ul>
<li>Adjusted the documation for the decimation tutorial so that the priority module is correctly initialized</li>
</ul>

</tr>

<tr valign=top><td><b>2.3.1</b> (2012/12/20,Rev.778)</td><td>

<b>Core</b>
<ul>
<li>Return vertex handles of newly added vertices in split and split_copy for faces when passing points instead of handles</li>
<li>Fixed copy and paste typo in split_copy for face handle</li>
<li>Replaced fabs by the std methods to fix errors when using norms with double vectors</li>
</ul>

<b>IO</b>
<ul>
<li>Fixed missing cast in importer which lead to problems when using different vector type. (Thanks to Mario Deuss for the fix)</li>
<li>Fixed bug in OBJ reader, where some faces could be missing(Thanks to Ian Kerr for the Fix)</li>
</ul>

<b>Documentation</b>
<ul>
<li>Fixed Decimater example</li>
<li>Improved Docs for is_boundary() functions</li>
</ul>

<b>Unittests</b>
<ul>
<li>Added unittest for vertexOHalfedge Iterator</li>
<li>Added unittests for boundary vertices and faces</li>
<li>Added test for VectorT abs function</li>
</ul>


</tr>


<tr valign=top><td><b>2.3</b> (2012/11/12,Rev.758)</td><td>

<b>Core</b>
<ul>
<li>New garbage collection function with possibility to update existing handles</li>
<li>Fixed crash in garbage collection, if certain status flags are not available (warns in debug mode!)</li>
<li>Fixed some gcc-4.7 incompatibilities</li>
<li>TriMesh::split now returns the handle of the new vertex</li>
<li>Fixed delete_face function, not mariking halfedges as deleted, if the edge gets deleted.(Thanks to Maxime Quiblier for the bug report