diff --git a/dw/fltkplatform.cc b/dw/fltkplatform.cc index 699c9e5..dd872a5 100644 --- a/dw/fltkplatform.cc +++ b/dw/fltkplatform.cc @@ -390,11 +390,11 @@ FltkPlatform::FltkResourceFactory::createLabelButtonResource (const char } core::ui::ComplexButtonResource * -FltkPlatform::FltkResourceFactory::createComplexButtonResource (std::unique_ptr< core::Widget - > widget, +FltkPlatform::FltkResourceFactory::createComplexButtonResource (core::Widget + *widget, bool relief) { - return new ui::FltkComplexButtonResource (platform, std::move( widget ), relief); + return new ui::FltkComplexButtonResource (platform, widget, relief); } core::ui::ListResource * diff --git a/dw/fltkplatform.hh b/dw/fltkplatform.hh index 0e79031..6395594 100644 --- a/dw/fltkplatform.hh +++ b/dw/fltkplatform.hh @@ -104,7 +104,7 @@ private: core::ui::LabelButtonResource *createLabelButtonResource (const char *label); core::ui::ComplexButtonResource * - createComplexButtonResource (std::unique_ptr< core::Widget > widget, bool relief); + createComplexButtonResource (core::Widget *widget, bool relief); core::ui::ListResource * createListResource (core::ui::ListResource::SelectionMode selectionMode, int rows); diff --git a/dw/fltkui.cc b/dw/fltkui.cc index fb4f839..42926a2 100644 --- a/dw/fltkui.cc +++ b/dw/fltkui.cc @@ -726,14 +726,14 @@ void FltkLabelButtonResource::setLabel (const char *label) // ---------------------------------------------------------------------- FltkComplexButtonResource::FltkComplexButtonResource (FltkPlatform *platform, - std::unique_ptr< dw::core::Widget - > widget, bool relief): + dw::core::Widget + *widget, bool relief): FltkSpecificResource (platform) { flatView = topView = NULL; this->relief = relief; FltkResource::init (platform); - ComplexButtonResource::init (std::move( widget )); + ComplexButtonResource::init (widget); } FltkComplexButtonResource::~FltkComplexButtonResource () diff --git a/dw/fltkui.hh b/dw/fltkui.hh index 1337dcf..e416683 100644 --- a/dw/fltkui.hh +++ b/dw/fltkui.hh @@ -5,8 +5,6 @@ # error Do not include this file directly, use "fltkcore.hh" instead. #endif -#include - #include #include #include @@ -286,7 +284,7 @@ protected: Fl_Widget *createNewWidget (core::Allocation *allocation); public: - FltkComplexButtonResource (FltkPlatform *platform, std::unique_ptr< dw::core::Widget > widget, + FltkComplexButtonResource (FltkPlatform *platform, dw::core::Widget *widget, bool relief); ~FltkComplexButtonResource (); }; diff --git a/dw/layout.cc b/dw/layout.cc index 6ba4e7c..82b0feb 100644 --- a/dw/layout.cc +++ b/dw/layout.cc @@ -327,8 +327,10 @@ Layout::~Layout () if (bgImage) bgImage->unref (); if (topLevel) { - detachWidget (topLevel.get()); - topLevel.reset(); + detachWidget (topLevel); + Widget *w = topLevel; + topLevel = NULL; + delete w; } delete queueResizeList; @@ -362,7 +364,7 @@ void Layout::detachWidget (Widget *widget) it->unref (); } -void Layout::addWidget (std::unique_ptr< Widget > widget) +void Layout::addWidget (Widget *widget) { if (topLevel) { MSG_WARN("widget already set\n"); @@ -372,20 +374,20 @@ void Layout::addWidget (std::unique_ptr< Widget > widget) // The toplevel widget always establishes a stacking context. It could // already be set in Widget::setStyle(). if (widget->stackingContextMgr == NULL && IMPL_POS) { - widget->stackingContextMgr = new StackingContextMgr (widget.get()); - DBG_OBJ_ASSOC (widget.get(), widget->stackingContextMgr); - widget->stackingContextWidget = widget.get(); + widget->stackingContextMgr = new StackingContextMgr (widget); + DBG_OBJ_ASSOC (widget, widget->stackingContextMgr); + widget->stackingContextWidget = widget; } - topLevel = std::move( widget ); - topLevel->layout = this; - topLevel->container = NULL; - DBG_OBJ_SET_PTR_O (topLevel.get(), "container", topLevel->container); + topLevel = widget; + widget->layout = this; + widget->container = NULL; + DBG_OBJ_SET_PTR_O (widget, "container", widget->container); queueResizeList->clear (); - topLevel->notifySetAsTopLevel (); + widget->notifySetAsTopLevel (); - findtextState.setWidget (topLevel.get()); + findtextState.setWidget (widget); canvasHeightGreater = false; DBG_OBJ_SET_SYM ("canvasHeightGreater", @@ -424,16 +426,18 @@ void Layout::removeWidget () updateCursor (); } -void Layout::setWidget (std::unique_ptr< Widget >widget) +void Layout::setWidget (Widget *widget) { DBG_OBJ_ASSOC_CHILD (widget); widgetAtPoint = NULL; if (topLevel) { - topLevel.reset(); + Widget *w = topLevel; + topLevel = NULL; + delete w; } textZone->zoneFree (); - addWidget (std::move(widget)); + addWidget (widget); updateCursor (); diff --git a/dw/layout.hh b/dw/layout.hh index 4f24c49..8389b3a 100644 --- a/dw/layout.hh +++ b/dw/layout.hh @@ -158,8 +158,7 @@ private: Platform *platform; View *view; - std::unique_ptr< Widget > topLevel; - Widget *widgetAtPoint; + Widget *topLevel, *widgetAtPoint; lout::container::typed::Vector *queueResizeList; /* The state, which must be projected into the view. */ @@ -282,8 +281,8 @@ public: lout::misc::ZoneAllocator *textZone; - void addWidget (std::unique_ptr< Widget > widget); - void setWidget (std::unique_ptr< Widget > widget); + void addWidget (Widget *widget); + void setWidget (Widget *widget); void attachView (View *view); void detachView (View *view); diff --git a/dw/ui.cc b/dw/ui.cc index 3364cc4..6bd2ad2 100644 --- a/dw/ui.cc +++ b/dw/ui.cc @@ -340,15 +340,15 @@ ComplexButtonResource::ComplexButtonResource () click_x = click_y = -1; } -void ComplexButtonResource::init (std::unique_ptr< Widget > widget) +void ComplexButtonResource::init (Widget *widget) { - childWidget = widget.get(); + childWidget = widget; /* FIXME: Buttons should not need a full Layout */ layout = new Layout (createPlatform (), false); setLayout (layout); DBG_OBJ_ASSOC_CHILD (layout); - layout->setWidget (std::move( widget )); + layout->setWidget (widget); layout->connect (&layoutReceiver); if (getEmbed ()) diff --git a/dw/ui.hh b/dw/ui.hh index ef73241..f7d0e5b 100644 --- a/dw/ui.hh +++ b/dw/ui.hh @@ -5,8 +5,6 @@ # error Do not include this file directly, use "core.hh" instead. #endif -#include - namespace dw { namespace core { @@ -435,7 +433,7 @@ protected: virtual int reliefXThickness () = 0; virtual int reliefYThickness () = 0; - void init (std::unique_ptr< Widget > widget); + void init (Widget *widget); public: ComplexButtonResource (); @@ -584,7 +582,7 @@ class ResourceFactory: public lout::object::Object public: virtual LabelButtonResource *createLabelButtonResource (const char *label) = 0; - virtual ComplexButtonResource *createComplexButtonResource (std::unique_ptr< Widget > widget, + virtual ComplexButtonResource *createComplexButtonResource (Widget *widget, bool relief) = 0; virtual ListResource *createListResource (ListResource::SelectionMode diff --git a/dw/widget.cc b/dw/widget.cc index 8874f70..79ea4be 100644 --- a/dw/widget.cc +++ b/dw/widget.cc @@ -214,7 +214,7 @@ void Widget::drawInterruption (View *view, Rectangle *area, DrawingContext *context) { Rectangle thisArea; - if (intersects (layout->topLevel.get(), context->getToplevelArea (), &thisArea)) + if (intersects (layout->topLevel, context->getToplevelArea (), &thisArea)) draw (view, &thisArea, context); context->addWidgetProcessedAsInterruption (this); diff --git a/src/form.cc b/src/form.cc index 5e7e7bd..f26bb5f 100644 --- a/src/form.cc +++ b/src/form.cc @@ -945,8 +945,7 @@ void Html_tag_open_button(DilloHtml *html, const char *tag, int tagsize) if (inp_type != DILLO_HTML_INPUT_UNKNOWN) { /* Render the button */ - std::unique_ptr< Widget > page; - Widget *page_p; + Widget *page; Embed *embed; const char *attrbuf; char *name, *value; @@ -960,18 +959,17 @@ void Html_tag_open_button(DilloHtml *html, const char *tag, int tagsize) /* We used to have Textblock (prefs.limit_text_width, ...) here, * but it caused 100% CPU usage. */ - page = std::make_unique< Textblock > (false, true); + page = new Textblock (false, true); page->setStyle (html->backgroundStyle ()); - page_p= page.get(); ResourceFactory *factory = HT2LT(html)->getResourceFactory(); - Resource *resource = factory->createComplexButtonResource(std::move( page ), true); + Resource *resource = factory->createComplexButtonResource(page, true); embed = new Embed(resource); // a_Dw_button_set_sensitive (DW_BUTTON (button), FALSE); HT2TB(html)->addWidget (embed, html->backgroundStyle ()); - S_TOP(html)->textblock = html->dw = page_p; + S_TOP(html)->textblock = html->dw = page; value = a_Html_get_attr_wdef(html, tag, tagsize, "value", NULL); name = a_Html_get_attr_wdef(html, tag, tagsize, "name", NULL); @@ -2044,12 +2042,11 @@ static Embed *Html_input_image(DilloHtml *html, const char *tag, int tagsize) // At this point, we know that Image->ir represents an image // widget. Notice that the order of the casts matters, because // of multiple inheritance. - std::unique_ptr< dw::Image > dwi; - dwi.reset( reinterpret_cast< dw::Image *>( (dw::core::ImgRenderer*)Image->img_rndr ) ); + dw::Image *dwi = (dw::Image*)(dw::core::ImgRenderer*)Image->img_rndr; dwi->setStyle (html->backgroundStyle ()); ResourceFactory *factory = HT2LT(html)->getResourceFactory(); ComplexButtonResource *complex_b_r = - factory->createComplexButtonResource(std::move( dwi ), false); + factory->createComplexButtonResource(dwi, false); button = new Embed(complex_b_r); HT2TB(html)->addWidget (button, html->style ()); } diff --git a/src/web.cc b/src/web.cc index 794c2f9..6fdb413 100644 --- a/src/web.cc +++ b/src/web.cc @@ -85,7 +85,7 @@ int a_Web_dispatch_by_type (const char *Type, DilloWeb *Web, dw->setStyle (styleEngine.style (Web->bw)); /* This method frees the old dw if any */ - layout->setWidget(std::unique_ptr< Widget >{ dw }); + layout->setWidget(dw); /* Set the page title with the bare filename (e.g. for images), * HTML pages with a tag will overwrite it later */