Delete part of merged mesh and get seg fault.

[Daniel Scott Matthews]

Try this:

Create a mesh cylinder and duplicate it, then move tool the copy +Z so
you have a gap between the two, then select both and apply mesh merge
followed by a bulge points. Select point selection tool and select the
top half of the top cylinder then apply delete (the icon with the
yellow lightning bolt), this will seg fault K-3D.

Here is the back-trace:

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x2b1728c3b250 (LWP 29614)]
0x00002b1738673c06 in
module::opengl::painters::point_painter::on_paint_mesh
(this=0x39b3c80, Mesh=@0x1f449a0, RenderState=@0x7fff87582e10,
ChangedSignal=@0x1fc0480)
    at /home/daniel/projects/k3d/modules/reference_opengl_painters/point_painter.cpp:81
81 k3d::gl::color3d(point_selection[point] ? selected_color : color);
(gdb) bt
#0  0x00002b1738673c06 in
module::opengl::painters::point_painter::on_paint_mesh
(this=0x39b3c80, Mesh=@0x1f449a0, RenderState=@0x7fff87582e10,
ChangedSignal=@0x1fc0480)
    at /home/daniel/projects/k3d/modules/reference_opengl_painters/point_painter.cpp:81
#1  0x00002b17386023fc in k3d::gl::mesh_painter::paint_mesh
(this=0x39b3c80, Mesh=@0x1f449a0, RenderState=@0x7fff87582e10,
ChangedSignal=@0x1fc0480) at
/home/daniel/projects/k3d/k3dsdk/mesh_painter_gl.h:57
#2  0x00002b1738650beb in
module::opengl::painters::multi_painter::on_paint_mesh
(this=0x39b4400, Mesh=@0x1f449a0, RenderState=@0x7fff87582e10,
ChangedSignal=@0x1fc0480)
    at /home/daniel/projects/k3d/modules/reference_opengl_painters/multi_painter.cpp:62
#3  0x00002b17386023fc in k3d::gl::mesh_painter::paint_mesh
(this=0x39b4400, Mesh=@0x1f449a0, RenderState=@0x7fff87582e10,
ChangedSignal=@0x1fc0480) at
/home/daniel/projects/k3d/k3dsdk/mesh_painter_gl.h:57
#4  0x00002b17397586cc in
module::mesh_instance::mesh_instance::on_gl_draw (this=0x1fbfcd0,
State=@0x7fff87583210) at
/home/daniel/projects/k3d/modules/mesh_instance/mesh_instance.cpp:154
#5  0x00002b1739758e25 in
k3d::gl::renderable<k3d::ri::renderable<k3d::mesh_selection_sink<k3d::parentable<k3d::transformable<k3d::node>
> > > >::gl_draw (this=0x1fbfcd0, State=@0x7fff87583210)
    at /home/daniel/projects/k3d/k3dsdk/renderable_gl.h:78
#6  0x00002b1733d77f84 in module::opengl::detail::draw::operator()
(this=0x7fff87583130, Object=0x1fbfcd0) at
/home/daniel/projects/k3d/modules/opengl/render_engine.cpp:121
#7  0x00002b1733d77fba in
std::for_each<std::_List_const_iterator<k3d::inode*>,
module::opengl::detail::draw> (__first={_M_node = 0x1796580},
__last={_M_node = 0x7fff875833a0}, __f=
      {m_state = @0x7fff87583210, m_node_selection = 0x2e72488}) at
/usr/include/c++/4.3/bits/stl_algo.h:3791
#8  0x00002b1733d8eef6 in
module::opengl::render_engine::render_viewport (this=0x2e65400,
Camera=@0x2e68288, PixelWidth=1277, PixelHeight=841,
ViewMatrix=0x3218550, ProjectionMatrix=0x32185d0, Viewport=0x3218650)
    at /home/daniel/projects/k3d/modules/opengl/render_engine.cpp:533
#9  0x00002b17294e46b0 in k3d::ngui::viewport::control::on_redraw
(this=0x3213130) at
/home/daniel/projects/k3d/k3dsdk/ngui/viewport.cpp:1349
#10 0x00002b17294eca4b in sigc::bound_mem_functor0<bool,
k3d::ngui::viewport::control>::operator() (this=0x32188e0) at
/usr/include/sigc++-2.0/sigc++/functors/mem_fun.h:1787
#11 0x00002b17294eca67 in
sigc::adaptor_functor<sigc::bound_mem_functor0<bool,
k3d::ngui::viewport::control> >::operator() (this=0x32188d8) at
/usr/include/sigc++-2.0/sigc++/adaptors/adaptor_trait.h:251
#12 0x00002b17294eca87 in sigc::hide_functor<-1,
sigc::bound_mem_functor0<bool, k3d::ngui::viewport::control>
>::operator()<_GdkEventExpose* const&> (this=0x32188d0) at
/usr/include/sigc++-2.0/sigc++/adaptors/hide.h:91
#13 0x00002b17294ecab2 in
sigc::internal::slot_call1<sigc::hide_functor<-1,
sigc::bound_mem_functor0<bool, k3d::ngui::viewport::control> >, bool,
_GdkEventExpose*>::call_it (rep=0x32188a0, a_1=@0x7fff87583758)
    at /usr/include/sigc++-2.0/sigc++/functors/slot.h:137
#14 0x00002b172ca413d2 in ?? () from /usr/lib/libgtkmm-2.4.so.1
#15 0x00002b172a93bdf8 in ?? () from /usr/lib/libgtk-x11-2.0.so.0
#16 0x00002b172488327d in g_closure_invoke () from /usr/lib/libgobject-2.0.so.0
#17 0x00002b17248993bc in ?? () from /usr/lib/libgobject-2.0.so.0
#18 0x00002b172489a2bd in g_signal_emit_valist () from
/usr/lib/libgobject-2.0.so.0
#19 0x00002b172489a953 in g_signal_emit () from /usr/lib/libgobject-2.0.so.0
#20 0x00002b172aa4409e in ?? () from /usr/lib/libgtk-x11-2.0.so.0
#21 0x00002b172a93563a in gtk_main_do_event () from /usr/lib/libgtk-x11-2.0.so.0
#22 0x00002b172b042cd5 in ?? () from /usr/lib/libgdk-x11-2.0.so.0
#23 0x00002b172b043261 in gdk_window_process_all_updates () from
/usr/lib/libgdk-x11-2.0.so.0
#24 0x00002b172b043289 in ?? () from /usr/lib/libgdk-x11-2.0.so.0
#25 0x00002b172b02704b in ?? () from /usr/lib/libgdk-x11-2.0.so.0
#26 0x00002b1724cfc20a in g_main_context_dispatch () from
/usr/lib/libglib-2.0.so.0
#27 0x00002b1724cff8e0 in ?? () from /usr/lib/libglib-2.0.so.0
#28 0x00002b1724cffdad in g_main_loop_run () from /usr/lib/libglib-2.0.so.0
#29 0x00002b172a935bc7 in gtk_main () from /usr/lib/libgtk-x11-2.0.so.0
#30 0x00002b1728db5da5 in
module::ngui::user_interface::start_event_loop (this=0x12d2e90) at
/home/daniel/projects/k3d/modules/ngui/user_interface.cpp:448
#31 0x000000000041e567 in k3d_main (raw_arguments=
          {<std::_Vector_base<std::basic_string<char,
std::char_traits<char>, std::allocator<char> >,
std::allocator<std::basic_string<char, std::char_traits<char>,
std::allocator<char> > > >> = {_M_impl =
{<std::allocator<std::basic_string<char, std::char_traits<char>,
std::allocator<char> > >> =
{<__gnu_cxx::new_allocator<std::basic_string<char,
std::char_traits<char>, std::allocator<char> > >> = {<No data
fields>}, <No data fields>}, _M_start = 0x7fff875846e0, _M_finish =
0x2b172665b558, _M_end_of_storage = 0x0}}, <No data fields>}) at
/home/daniel/projects/k3d/application/k3d_main.cpp:823
#32 0x0000000000427c50 in main (argc=8, argv=0x7fff87584808) at
/home/daniel/projects/k3d/application/main.cpp:32

------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
K3d-development mailing list
K3d-development@...
https://lists.sourceforge.net/lists/listinfo/k3d-development

[bART Janssens-2]

On Thu, Oct 1, 2009 at 7:42 AM, Daniel Scott Matthews
<dsmatthews@...> wrote:
> Create a mesh cylinder and duplicate it, then move tool the copy +Z so
> you have a gap between the two, then select both and apply mesh merge
> followed by a bulge points. Select point selection tool and select the
> top half of the top cylinder then apply delete (the icon with the
> yellow lightning bolt), this will seg fault K-3D.

The delete tool is not synced with the latest changes to the mesh
primitive structure. I'll look at this on the train tomorrow ;)

Cheers,

--
Bart

------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
K3d-development mailing list
K3d-development@...
https://lists.sourceforge.net/lists/listinfo/k3d-development

[bART Janssens-2]

On Thu, Oct 1, 2009 at 7:42 AM, Daniel Scott Matthews
<dsmatthews@...> wrote:
> Create a mesh cylinder and duplicate it, then move tool the copy +Z so
> you have a gap between the two, then select both and apply mesh merge
> followed by a bulge points. Select point selection tool and select the
> top half of the top cylinder then apply delete (the icon with the
> yellow lightning bolt), this will seg fault K-3D.

Should be fixed in
http://code.google.com/p/k3d-bart/source/detail?r=228e4bf0d9d34bc3eeafe0d4647ff67bc59b6d9a

Cheers,

--
Bart

------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
K3d-development mailing list
K3d-development@...
https://lists.sourceforge.net/lists/listinfo/k3d-development

[Daniel Scott Matthews]

On Tue, Oct 6, 2009 at 6:34 AM, Bart Janssens <bart.janssens@...> wrote:

> On Thu, Oct 1, 2009 at 7:42 AM, Daniel Scott Matthews
> <dsmatthews@...> wrote:
>> Create a mesh cylinder and duplicate it, then move tool the copy +Z so
>> you have a gap between the two, then select both and apply mesh merge
>> followed by a bulge points. Select point selection tool and select the
>> top half of the top cylinder then apply delete (the icon with the
>> yellow lightning bolt), this will seg fault K-3D.
>
> Should be fixed in
> http://code.google.com/p/k3d-bart/source/detail?r=228e4bf0d9d34bc3eeafe0d4647ff67bc59b6d9a
>

Thanks, I'll test that once Tim has merged it into the main repo.

------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
K3d-development mailing list
K3d-development@...
https://lists.sourceforge.net/lists/listinfo/k3d-development

[Timothy M. Shead]

Daniel Scott Matthews wrote:

> On Tue, Oct 6, 2009 at 6:34 AM, Bart Janssens <bart.janssens@...> wrote:
>> On Thu, Oct 1, 2009 at 7:42 AM, Daniel Scott Matthews
>> <dsmatthews@...> wrote:
>>> Create a mesh cylinder and duplicate it, then move tool the copy +Z so
>>> you have a gap between the two, then select both and apply mesh merge
>>> followed by a bulge points. Select point selection tool and select the
>>> top half of the top cylinder then apply delete (the icon with the
>>> yellow lightning bolt), this will seg fault K-3D.
>> Should be fixed in
>> http://code.google.com/p/k3d-bart/source/detail?r=228e4bf0d9d34bc3eeafe0d4647ff67bc59b6d9a
>>
> Thanks, I'll test that once Tim has merged it into the main repo.

Merged to SF.

Cheers,
Tim

[tshead.vcf]

---

begin:vcard
fn:Timothy Shead
n:Shead;Timothy
org:www.k-3d.org
email;internet:tshead@...
title:Founder
x-mozilla-html:FALSE
version:2.1
end:vcard

---

------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
K3d-development mailing list
K3d-development@...
https://lists.sourceforge.net/lists/listinfo/k3d-development

[Daniel Scott Matthews]

On Tue, Oct 6, 2009 at 2:13 PM, Timothy M. Shead <tshead@...> wrote:

> Daniel Scott Matthews wrote:
>>
>> On Tue, Oct 6, 2009 at 6:34 AM, Bart Janssens <bart.janssens@...>
>> wrote:
>>>
>>> On Thu, Oct 1, 2009 at 7:42 AM, Daniel Scott Matthews
>>> <dsmatthews@...> wrote:
>>>>
>>>> Create a mesh cylinder and duplicate it, then move tool the copy +Z so
>>>> you have a gap between the two, then select both and apply mesh merge
>>>> followed by a bulge points. Select point selection tool and select the
>>>> top half of the top cylinder then apply delete (the icon with the
>>>> yellow lightning bolt), this will seg fault K-3D.
>>>
>>> Should be fixed in
>>>
>>> http://code.google.com/p/k3d-bart/source/detail?r=228e4bf0d9d34bc3eeafe0d4647ff67bc59b6d9a
>>>
>> Thanks, I'll test that once Tim has merged it into the main repo.
>
> Merged to SF.
>

and built and tested on 64bit Linux. Thanks.

------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
K3d-development mailing list
K3d-development@...
https://lists.sourceforge.net/lists/listinfo/k3d-development