The new disassembler has been converted incompletely

Yes, I'm aware of that. A bit of information needs to be returned by the eAccelerator code, but most stuff can be done in the php page.

Can you take a look at the current version? I've added source code and changed the looks a bit. The compiled variables and color coding haven't been added yet.

<?php
    if (!isset($_GET['file']) && is_file($_GET['file'])) {
        die('File argument not given!');
    }
?>

Should be

<?php
    if (!isset($_GET['file']) || !is_file($_GET['file'])) {
        die('File argument not given!');
    }
?>

I guess.

Regards,

Christian Stadler

And you should replace

<?php
echo "<div style=\"text-align: left; widht: 800px\"><ul>\n";
?>

with

<?php
echo "<div style=\"text-align: left; width: 800px\"><ul>\n";
?>

twice. You spelled width wrong.

in print_method:

shouldn't

<?php
echo "<h4>Method: $name</h4>";
?>

be

<?php
echo "<h2>Method: $name</h2>";
?>

?

The following doesn't work quite right:

    <?php
        $count = count($op_array);
        $line = 0;
        global $source;
        $last_line = 0;
        for ($i = 0; $i < $count; ++$i) {
            $curr_line =  $op_array[$i]['lineno'];
            /* find the next line that differs, but only when the start line differs from the previous */
            $print = $line;
            if ($last_line < $curr_line) {
                for ($j = $i + 1; $j < $count; ++$j) {
                    if ($op_array[$j]['lineno'] > $curr_line) {
                        $print = $op_array[$j]['lineno'] - 1;
                        break;
                    }
                }
            }
            $code = '';
            while($line < $print) {
                $code .= sprintf("%03d: %s\n", ($line + 1), $source[$line]);
                ++$line;
            }
            if ($code != '') {
                echo "<tr>\n";
//                echo '<td  class="source" colspan="6"><pre>' . highlight_string($code, true) . "</pre></td>\n";
                echo '<td  class="source" colspan="6"><pre>' . $code . "</pre></td>\n";
                echo "</tr>\n";
            }
    ?>

At the beginning everything looks fine (the correct code is above the oplines), but later on the oplines respectively the code are misplaced.

For example, when you display dasm.php with itself to problem starts around line 96 (Opline 129). The fact, that $last_line never changes may be one of the causes, but I'm too tired to do more debugging.

PS: dasm.php should have HTTP_AUTH too. Otherwise anyone could access it and read files, he shouldn't read.

I know putting the code between the opcodes goes wrong sometimes. I've been debugging this and it looks like the opcodes are placed on the lines the zend_engine put's in the op_array. The old disassembler also does this to. Am I wrong here?

I'll see, if I find a solution now.

btw: I've added $op_array[$i]lineno? to the table (column-title: Line) perhaps this could be the default?

PS: How about adding the method-list to print_layout and place it on the top of every page of dasm.php? This would make print_class redundant and you could navigate through the script layout on every page.

I got it (I think). See the attached patch.

Additionally it adds $op_array[$i]lineno? to the table and it replaces the 'and' with an 'or' in the check for the file argument.

Regards,

Christian Stadler

btw: If you want the codelines to be highlighted, you should take a look at http://aidanlister.com/repos/v/PHP_Highlight.php

I've included the patch in r198. You can implement the menu stuff and the source highlighting. I'll include it, it would be great to have those things.

You can implement the menu stuff and the source highlighting. I'll include it, it would be great to have those things.

Done (See the attachment). And I've removed some redundant code.

btw: You should mention Aidan Lister <aidan [ett] php [dutt] net> in the AUTHORS-file. perhaps under a new section called 'Other contributors' for his great highlighter :)

This patch is added in revision [206] It's a very nice improvement! Is the new disassembler complete now? Or do you need some other stuff to?

Well, coloring and compiled variables are still missing, but that isn't so important and could be solved in a future Release.

Changing milestone and priority.

Singles Kontakte

Dating Guide

Adult Chat Kontakte

Adult Dating Kontakte

Dating Online Guide

Flirt Liebe Kontakte