WHILE in context where it is not needed at all?

[GoogleCodeExporter]

3 methods are using while command, but are those needed on this context at all?

    function get_parent() {

        $p = $this->_frame->get_parent ();

        if ($p && $deco = $p->get_decorator ()) {
            while ( $tmp = $deco->get_decorator () )
                $deco = $tmp;
            return $deco;
        } else if ($p)
            return $p;
        else
            return null;
    }

    function get_first_child() {
        $c = $this->_frame->get_first_child ();
        if ($c && $deco = $c->get_decorator ()) {
            while ( $tmp = $deco->get_decorator () )
                $deco = $tmp;
            return $deco;
        } else if ($c)
            return $c;
        else
            return null;
    }

    function get_last_child() {
        $c = $this->_frame->get_last_child ();
        if ($c && $deco = $c->get_decorator ()) {
            while ( $tmp = $deco->get_decorator () )
                $deco = $tmp;
            return $deco;
        } else if ($c)
            return $c;
        else
            return null;
    }

Original issue reported on code.google.com by holyfi...@devbau.com on 8 Sep 2009 at 10:13

[GoogleCodeExporter]

version: dompdf 0.5.1

Original comment by holyfi...@devbau.com on 8 Sep 2009 at 10:15

[GoogleCodeExporter]

File: frame_decorator.cls.php

Original comment by holyfi...@devbau.com on 8 Sep 2009 at 10:17

[GoogleCodeExporter]

Why wouldn't it be needed? The code here is to find the furthest decorator.

Original comment by fabien.menager on 27 Jul 2010 at 8:21

• Changed state: Invalid

[GoogleCodeExporter]

Original comment by eclecticgeek on 30 May 2013 at 5:15

• Added labels: Restrict-AddIssueComment-Commit