Re: more B::Concise stuff (PATCH - updated)
[p5sagit/p5-mst-13.2.git] / ext / B / B / Concise.pm
index cb40021..eb9398a 100644 (file)
@@ -14,11 +14,16 @@ use warnings; # uses #3 and #4, since warnings uses Carp
 
 use Exporter (); # use #5
 
-our $VERSION   = "0.60";
+our $VERSION   = "0.62";
 our @ISA       = qw(Exporter);
-our @EXPORT_OK = qw(set_style set_style_standard add_callback
-                   concise_subref concise_cv concise_main
-                   add_style walk_output);
+our @EXPORT_OK = qw( set_style set_style_standard add_callback
+                    concise_subref concise_cv concise_main
+                    add_style walk_output compile reset_sequence );
+our %EXPORT_TAGS =
+    ( io       => [qw( walk_output compile reset_sequence )],
+      style    => [qw( add_style set_style_standard )],
+      cb       => [qw( add_callback )],
+      mech     => [qw( concise_subref concise_cv concise_main )],  );
 
 # use #6
 use B qw(class ppname main_start main_root main_cv cstring svref_2object
@@ -33,8 +38,8 @@ my %style =
     "#class pp_#name"],
    "concise" =>
    ["#hyphseq2 (*(   (x( ;)x))*)<#classsym> "
-    . "#exname#arg(?([#targarglife])?)~#flags(?(/#private)?)(x(;~->#next)x)\n",
-    "  (*(    )*)     goto #seq\n",
+    . "#exname#arg(?([#targarglife])?)~#flags(?(/#private)?)(x(;~->#next)x)\n"
+    , "  (*(    )*)     goto #seq\n",
     "(?(<#seq>)?)#exname#arg(?([#targarglife])?)"],
    "linenoise" =>
    ["(x(;(*( )*))x)#noise#arg(?([#targarg])?)(x( ;\n)x)",
@@ -52,14 +57,34 @@ my %style =
             $ENV{B_CONCISE_TREE_FORMAT}],
   );
 
-my($format, $gotofmt, $treefmt);
+# Renderings, ie how Concise prints, is controlled by these vars
+# primary:
+our $stylename;                # selects current style from %style
+my $order = "basic";   # how optree is walked & printed: basic, exec, tree
+
+# rendering mechanics:
+# these 'formats' are the line-rendering templates
+# they're updated from %style when $stylename changes
+my ($format, $gotofmt, $treefmt);
+
+# lesser players:
+my $base = 36;         # how <sequence#> is displayed
+my $big_endian = 1;    # more <sequence#> display
+my $tree_style = 0;    # tree-order details
+my $banner = 1;                # print banner before optree is traversed
+my $do_main = 0;       # force printing of main routine
+
+# another factor: can affect all styles!
+our @callbacks;                # allow external management
+
+set_style_standard("concise");
+
 my $curcv;
 my $cop_seq_base;
-my @callbacks;
-my $stylename;
 
 sub set_style {
     ($format, $gotofmt, $treefmt) = @_;
+    #warn "set_style: deprecated, use set_style_standard instead\n"; # someday
     die "expecting 3 style-format args\n" unless @_ == 3;
 }
 
@@ -69,10 +94,11 @@ sub add_style {
        if exists $style{$newstyle};
     die "expecting 3 style-format args\n" unless @args == 3;
     $style{$newstyle} = [@args];
+    $stylename = $newstyle; # update rendering state
 }
 
 sub set_style_standard {
-    ($stylename) = @_;
+    ($stylename) = @_; # update rendering state
     die "err: style '$stylename' unknown\n" unless exists $style{$stylename};
     set_style(@{$style{$stylename}});
 }
@@ -82,29 +108,51 @@ sub add_callback {
 }
 
 # output handle, used with all Concise-output printing
-our $walkHandle = \*STDOUT;    # public for your convenience
+our $walkHandle;       # public for your convenience
+BEGIN { $walkHandle = \*STDOUT }
 
 sub walk_output { # updates $walkHandle
     my $handle = shift;
+    return $walkHandle unless $handle; # allow use as accessor
+
     if (ref $handle eq 'SCALAR') {
+       require Config;
+       die "no perlio in this build, can't call walk_output (\\\$scalar)\n"
+           unless $Config::Config{useperlio};
        # in 5.8+, open(FILEHANDLE,MODE,REFERENCE) writes to string
-       open my $tmp, '>', $handle;     # but cant re-set an existing filehandle
+       open my $tmp, '>', $handle;     # but cant re-set existing STDOUT
        $walkHandle = $tmp;             # so use my $tmp as intermediate var
-       return;
+       return $walkHandle;
     }
-    $walkHandle = $handle;
-    my $iotype = ref $walkHandle;
+    my $iotype = ref $handle;
     die "expecting argument/object that can print\n"
-       unless $iotype eq 'GLOB' or $iotype and $walkHandle->can('print');
+       unless $iotype eq 'GLOB' or $iotype and $handle->can('print');
+    $walkHandle = $handle;
 }
 
 sub concise_subref {
     my($order, $coderef) = @_;
     my $codeobj = svref_2object($coderef);
-    die "err: not a coderef: $coderef\n" unless ref $codeobj eq 'B::CV';#CODE';
+
+    return concise_stashref(@_)        
+       unless ref $codeobj eq 'B::CV';
     concise_cv_obj($order, $codeobj);
 }
 
+sub concise_stashref {
+    my($order, $h) = @_;
+    foreach my $k (sort keys %$h) {
+       local *s = $h->{$k};
+       my $coderef = *s{CODE} or next;
+       reset_sequence();
+       print "FUNC: ", *s, "\n";
+       my $codeobj = svref_2object($coderef);
+       next unless ref $codeobj eq 'B::CV';
+       eval { concise_cv_obj($order, $codeobj) }
+       or warn "err $@ on $codeobj";
+    }
+}
+
 # This should have been called concise_subref, but it was exported
 # under this name in versions before 0.56
 sub concise_cv { concise_subref(@_); }
@@ -164,27 +212,23 @@ my @tree_decorations =
    ["  ", map("$start_sym$_$end_sym", "qq", "wq", "tq", "x ", "mq", "q"), 1],
    [" ", map("$start_sym$_$end_sym", "q", "w", "t", "x", "m"), "", 0],
   );
-my $tree_style = 0;
-
-my $base = 36;
-my $big_endian = 1;
 
-my $order = "basic";
 
-set_style_standard("concise");
-
-sub compile {
+sub compileOpts {
+    # set rendering state from options and args
     my @options = grep(/^-/, @_);
     my @args = grep(!/^-/, @_);
-    my $do_main = 0;
     for my $o (@options) {
+       # mode/order
        if ($o eq "-basic") {
            $order = "basic";
        } elsif ($o eq "-exec") {
            $order = "exec";
        } elsif ($o eq "-tree") {
            $order = "tree";
-       } elsif ($o eq "-compact") {
+       }
+       # tree-specific
+       elsif ($o eq "-compact") {
            $tree_style |= 1;
        } elsif ($o eq "-loose") {
            $tree_style &= ~1;
@@ -192,67 +236,89 @@ sub compile {
            $tree_style |= 2;
        } elsif ($o eq "-ascii") {
            $tree_style &= ~2;
-       } elsif ($o eq "-main") {
-           $do_main = 1;
-       } elsif ($o =~ /^-base(\d+)$/) {
+       }
+       # sequence numbering
+       elsif ($o =~ /^-base(\d+)$/) {
            $base = $1;
        } elsif ($o eq "-bigendian") {
            $big_endian = 1;
        } elsif ($o eq "-littleendian") {
            $big_endian = 0;
-       } elsif (exists $style{substr($o, 1)}) {
+       }
+       elsif ($o eq "-nobanner") {
+           $banner = 0;
+       } elsif ($o eq "-banner") {
+           $banner = 1;
+       }
+       elsif ($o eq "-main") {
+           $do_main = 1;
+       } elsif ($o eq "-nomain") {
+           $do_main = 0;
+       }
+       # line-style options
+       elsif (exists $style{substr($o, 1)}) {
            $stylename = substr($o, 1);
-           set_style(@{$style{$stylename}});
+           set_style_standard($stylename);
        } else {
            warn "Option $o unrecognized";
        }
     }
+    return (@args);
+}
+
+sub compile {
+    my (@args) = compileOpts(@_);
     return sub {
-       if (@args) {
-           for my $objname (@args) {
-               if ($objname eq "BEGIN") {
-                   concise_specials("BEGIN", $order,
-                                    B::begin_av->isa("B::AV") ?
-                                    B::begin_av->ARRAY : ());
-               } elsif ($objname eq "INIT") {
-                   concise_specials("INIT", $order,
-                                    B::init_av->isa("B::AV") ?
-                                    B::init_av->ARRAY : ());
-               } elsif ($objname eq "CHECK") {
-                   concise_specials("CHECK", $order,
-                                    B::check_av->isa("B::AV") ?
-                                    B::check_av->ARRAY : ());
-               } elsif ($objname eq "END") {
-                   concise_specials("END", $order,
-                                    B::end_av->isa("B::AV") ?
-                                    B::end_av->ARRAY : ());
+       my @newargs = compileOpts(@_); # accept new rendering options
+       warn "disregarding non-options: @newargs\n" if @newargs;
+
+       for my $objname (@args) {
+           
+           if ($objname eq "BEGIN") {
+               concise_specials("BEGIN", $order,
+                              B::begin_av->isa("B::AV") ?
+                              B::begin_av->ARRAY : ());
+           } elsif ($objname eq "INIT") {
+               concise_specials("INIT", $order,
+                              B::init_av->isa("B::AV") ?
+                              B::init_av->ARRAY : ());
+           } elsif ($objname eq "CHECK") {
+               concise_specials("CHECK", $order,
+                              B::check_av->isa("B::AV") ?
+                              B::check_av->ARRAY : ());
+           } elsif ($objname eq "END") {
+               concise_specials("END", $order,
+                                     B::end_av->isa("B::AV") ?
+                              B::end_av->ARRAY : ());
+           }
+           else {
+               # convert function names to subrefs
+               my $objref;
+               if (ref $objname) {
+                   print $walkHandle "B::Concise::compile($objname)\n"
+                       if $banner;
+                   $objref = $objname;
                } else {
-                   # convert function names to subrefs
-                   my $objref;
-                   if (ref $objname) {
-                       print $walkHandle "B::Concise::compile($objname)\n";
-                       $objref = $objname;
-                   } else {
-                       $objname = "main::" . $objname unless $objname =~ /::/;
-                       print $walkHandle "$objname:\n";
-                       no strict 'refs';
-                       die "err: unknown function ($objname)\n"
-                           unless *{$objname}{CODE};
-                       $objref = \&$objname;
-                   }
-                   concise_subref($order, $objref);
+                   $objname = "main::" . $objname unless $objname =~ /::/;
+                   print $walkHandle "$objname:\n";
+                   no strict 'refs';
+                   die "err: unknown function ($objname)\n"
+                       unless *{$objname}{CODE};
+                   $objref = \&$objname;
                }
+               concise_subref($order, $objref);
            }
        }
        if (!@args or $do_main) {
            print $walkHandle "main program:\n" if $do_main;
            concise_main($order);
        }
+       return @args;   # something
     }
 }
 
 my %labels;
-my $lastnext;
+my $lastnext;  # remembers op-chain, used to insert gotos
 
 my %opclass = ('OP' => "0", 'UNOP' => "1", 'BINOP' => "2", 'LOGOP' => "|",
               'LISTOP' => "@", 'PMOP' => "/", 'SVOP' => "\$", 'GVOP' => "*",
@@ -309,6 +375,7 @@ sub reset_sequence {
     # reset the sequence
     %sequence_num = ();
     $seq_max = 1;
+    $lastnext = 0;
 }
 
 sub seq {
@@ -410,19 +477,35 @@ sub sequence {
     }
 }
 
-sub fmt_line {
-    my($hr, $text, $level) = @_;
-    return '' if $hr->{SKIP};  # another way to suppress lines of output
+sub fmt_line {    # generate text-line for op.
+    my($hr, $op, $text, $level) = @_;
+
+    $_->($hr, $op, \$text, \$level, $stylename) for @callbacks;
+
+    return '' if $hr->{SKIP};  # suppress line if a callback said so
+    return '' if $hr->{goto} and $hr->{goto} eq '-';   # no goto nowhere
 
+    # spec: (?(text1#varText2)?)
     $text =~ s/\(\?\(([^\#]*?)\#(\w+)([^\#]*?)\)\?\)/
        $hr->{$2} ? $1.$hr->{$2}.$3 : ""/eg;
 
+    # spec: (x(exec_text;basic_text)x)
     $text =~ s/\(x\((.*?);(.*?)\)x\)/$order eq "exec" ? $1 : $2/egs;
+
+    # spec: (*(text)*)
     $text =~ s/\(\*\(([^;]*?)\)\*\)/$1 x $level/egs;
+
+    # spec: (*(text1;text2)*)
     $text =~ s/\(\*\((.*?);(.*?)\)\*\)/$1 x ($level - 1) . $2 x ($level>0)/egs;
-    $text =~ s/#([a-zA-Z]+)(\d+)/sprintf("%-$2s", $hr->{$1})/eg;
-    $text =~ s/#([a-zA-Z]+)/$hr->{$1}/eg;
-    $text =~ s/[ \t]*~+[ \t]*/ /g;
+
+    # convert #Var to tag=>val form: Var\t#var
+    $text =~ s/\#([A-Z][a-z]+)(\d+)?/\t\u$1\t\L#$1$2/gs;
+
+    # spec: #varN
+    $text =~ s/\#([a-zA-Z]+)(\d+)/sprintf("%-$2s", $hr->{$1})/eg;
+
+    $text =~ s/\#([a-zA-Z]+)/$hr->{$1}/eg;     # populate #var's
+    $text =~ s/[ \t]*~+[ \t]*/ /g;             # squeeze tildes
     chomp $text;
     return "$text\n" if $text ne "";
     return $text; # suppress empty lines
@@ -536,7 +619,10 @@ sub concise_sv {
        } elsif (class($sv) eq "HV") {
            $hr->{svval} .= 'HASH';
        }
-       return $hr->{svclass} . " " .  $hr->{svval};
+
+       $hr->{svval} = 'undef' unless defined $hr->{svval};
+       my $out = $hr->{svclass};
+       return $out .= " $hr->{svval}" ; 
     }
 }
 
@@ -669,16 +755,18 @@ sub concise_op {
     $h{typenum} = $op->type;
     $h{noise} = $linenoise[$op->type];
 
-    $_->(\%h, $op, \$format, \$level, $stylename) for @callbacks;
-    return fmt_line(\%h, $format, $level);
+    return fmt_line(\%h, $op, $format, $level);
 }
 
 sub B::OP::concise {
     my($op, $level) = @_;
     if ($order eq "exec" and $lastnext and $$lastnext != $$op) {
-       my $h = {"seq" => seq($lastnext), "class" => class($lastnext),
-                "addr" => sprintf("%#x", $$lastnext)};
-       print $walkHandle fmt_line($h, $gotofmt, $level+1);
+       # insert a 'goto' line
+       my $synth = {"seq" => seq($lastnext), "class" => class($lastnext),
+                    "addr" => sprintf("%#x", $$lastnext),
+                    "goto" => seq($lastnext), # simplify goto '-' removal
+            };
+       print $walkHandle fmt_line($synth, $op, $gotofmt, $level+1);
     }
     $lastnext = $op->next;
     print $walkHandle concise_op($op, $level, $format);
@@ -700,12 +788,15 @@ sub b_terse {
     $curcv = main_cv unless $curcv;
 
     if ($order eq "exec" and $lastnext and $$lastnext != $$op) {
+       # insert a 'goto'
        my $h = {"seq" => seq($lastnext), "class" => class($lastnext),
                 "addr" => sprintf("%#x", $$lastnext)};
-       print fmt_line($h, $style{"terse"}[1], $level+1);
+       print # $walkHandle
+           fmt_line($h, $op, $style{"terse"}[1], $level+1);
     }
     $lastnext = $op->next;
-    print concise_op($op, $level, $style{"terse"}[0]);
+    print # $walkHandle 
+       concise_op($op, $level, $style{"terse"}[0]);
 }
 
 sub tree {
@@ -805,8 +896,8 @@ sophisticated and flexible.
 
 =head1 EXAMPLE
 
-Here's is a short example of output, using the default formatting
-conventions :
+Here's is a short example of output (aka 'rendering'), using the
+default formatting conventions :
 
     % perl -MO=Concise -e '$a = $b + 42'
     8  <@> leave[1 ref] vKP/REFC ->(end)
@@ -820,7 +911,7 @@ conventions :
     -        <1> ex-rv2sv sKRM*/1 ->7
     6           <$> gvsv(*a) s ->7
 
-Each line corresponds to an operator. Null ops appear as C<ex-opname>,
+Each line corresponds to an opcode. Null ops appear as C<ex-opname>,
 where I<opname> is the op that has been optimized away by perl.
 
 The number on the first row indicates the op's sequence number. It's
@@ -850,6 +941,16 @@ including use'd or require'd files) is printed. Passing C<BEGIN>,
 C<CHECK>, C<INIT>, or C<END> will cause all of the corresponding
 special blocks to be printed.
 
+Options affect how things are rendered (ie printed).  They're presented
+here by their visual effect, 1st being strongest.  They're grouped
+according to how they interrelate; within each group the options are
+mutually exclusive (unless otherwise stated).
+
+=head2 Options for Opcode Ordering
+
+These options control the 'vertical display' of opcodes.  The display
+'order' is also called 'mode' elsewhere in this document.
+
 =over 4
 
 =item B<-basic>
@@ -876,6 +977,50 @@ at the left and 'left-to-right' order of children transformed into
 it isn't suitable for large programs (unless you have a very wide
 terminal).
 
+=back
+
+=head2 Options for Line-Style
+
+These options select the line-style (or just style) used to render
+each opcode, and dictates what info is actually printed into each line.
+
+=over 4
+
+=item B<-concise>
+
+Use the author's favorite set of formatting conventions. This is the
+default, of course.
+
+=item B<-terse>
+
+Use formatting conventions that emulate the output of B<B::Terse>. The
+basic mode is almost indistinguishable from the real B<B::Terse>, and the
+exec mode looks very similar, but is in a more logical order and lacks
+curly brackets. B<B::Terse> doesn't have a tree mode, so the tree mode
+is only vaguely reminiscent of B<B::Terse>.
+
+=item B<-linenoise>
+
+Use formatting conventions in which the name of each OP, rather than being
+written out in full, is represented by a one- or two-character abbreviation.
+This is mainly a joke.
+
+=item B<-debug>
+
+Use formatting conventions reminiscent of B<B::Debug>; these aren't
+very concise at all.
+
+=item B<-env>
+
+Use formatting conventions read from the environment variables
+C<B_CONCISE_FORMAT>, C<B_CONCISE_GOTO_FORMAT>, and C<B_CONCISE_TREE_FORMAT>.
+
+=back
+
+=head2 Options for tree-specific formatting
+
+=over 4
+
 =item B<-compact>
 
 Use a tree format in which the minimum amount of space is used for the
@@ -900,10 +1045,13 @@ look as clean as the VT100 characters, but they'll work with almost any
 terminal (or the horizontal scrolling mode of less(1)) and are suitable
 for text documentation or email. This is the default.
 
-=item B<-main>
+=back
 
-Include the main program in the output, even if subroutines were also
-specified.
+These are pairwise exclusive, i.e. compact or loose, vt or ascii.
+
+=head2 Options controlling sequence numbering
+
+=over 4
 
 =item B<-base>I<n>
 
@@ -919,47 +1067,122 @@ usual convention for Arabic numerals, and the default.
 
 =item B<-littleendian>
 
-Print seqence numbers with the least significant digit first.
+Print seqence numbers with the least significant digit first.  This is
+obviously mutually exclusive with bigendian.
 
-=item B<-concise>
+=back
 
-Use the author's favorite set of formatting conventions. This is the
-default, of course.
+=head2 Other options
 
-=item B<-terse>
+These are pairwise exclusive.
 
-Use formatting conventions that emulate the output of B<B::Terse>. The
-basic mode is almost indistinguishable from the real B<B::Terse>, and the
-exec mode looks very similar, but is in a more logical order and lacks
-curly brackets. B<B::Terse> doesn't have a tree mode, so the tree mode
-is only vaguely reminiscient of B<B::Terse>.
+=over 4
 
-=item B<-linenoise>
+=item B<-main>
 
-Use formatting conventions in which the name of each OP, rather than being
-written out in full, is represented by a one- or two-character abbreviation.
-This is mainly a joke.
+Include the main program in the output, even if subroutines were also
+specified.  This rendering is normally suppressed when a subroutine
+name or reference is given.
 
-=item B<-debug>
+=item B<-nomain>
 
-Use formatting conventions reminiscient of B<B::Debug>; these aren't
-very concise at all.
+This restores the default behavior after you've changed it with '-main'
+(it's not normally needed).  If no subroutine name/ref is given, main is
+rendered, regardless of this flag.
 
-=item B<-env>
+=item B<-nobanner>
 
-Use formatting conventions read from the environment variables
-C<B_CONCISE_FORMAT>, C<B_CONCISE_GOTO_FORMAT>, and C<B_CONCISE_TREE_FORMAT>.
+Renderings usually include a banner line identifying the function name
+or stringified subref.  This suppresses the printing of the banner.
+
+TBC: Remove the stringified coderef; while it provides a 'cookie' for
+each function rendered, the cookies used should be 1,2,3.. not a
+random hex-address.  It also complicates string comparison of two
+different trees.
+
+=item B<-banner>
+
+restores default banner behavior.
+
+=item B<-banneris> => subref
+
+TBC: a hookpoint (and an option to set it) for a user-supplied
+function to produce a banner appropriate for users needs.  It's not
+ideal, because the rendering-state variables, which are a natural
+candidate for use in concise.t, are unavailable to the user.
 
 =back
 
+=head2 Option Stickiness
+
+If you invoke Concise more than once in a program, you should know that
+the options are 'sticky'.  This means that the options you provide in
+the first call will be remembered for the 2nd call, unless you
+re-specify or change them.
+
+=head1 ABBREVIATIONS
+
+The concise style uses symbols to convey maximum info with minimal
+clutter (like hex addresses).  With just a little practice, you can
+start to see the flowers, not just the branches, in the trees.
+
+=head2 OP class abbreviations
+
+These symbols appear before the op-name, and indicate the
+B:: namespace that represents the ops in your Perl code.
+
+    0      OP (aka BASEOP)  An OP with no children
+    1      UNOP             An OP with one child
+    2      BINOP            An OP with two children
+    |      LOGOP            A control branch OP
+    @      LISTOP           An OP that could have lots of children
+    /      PMOP             An OP with a regular expression
+    $      SVOP             An OP with an SV
+    "      PVOP             An OP with a string
+    {      LOOP             An OP that holds pointers for a loop
+    ;      COP              An OP that marks the start of a statement
+    #      PADOP            An OP with a GV on the pad
+
+=head2 OP flags abbreviations
+
+These symbols represent various flags which alter behavior of the
+opcode, sometimes in opcode-specific ways.
+
+    v      OPf_WANT_VOID    Want nothing (void context)
+    s      OPf_WANT_SCALAR  Want single value (scalar context)
+    l      OPf_WANT_LIST    Want list of any length (list context)
+    K      OPf_KIDS         There is a firstborn child.
+    P      OPf_PARENS       This operator was parenthesized.
+                             (Or block needs explicit scope entry.)
+    R      OPf_REF          Certified reference.
+                             (Return container, not containee).
+    M      OPf_MOD          Will modify (lvalue).
+    S      OPf_STACKED      Some arg is arriving on the stack.
+    *      OPf_SPECIAL      Do something weird for this op (see op.h)
+
 =head1 FORMATTING SPECIFICATIONS
 
-For each general style ('concise', 'terse', 'linenoise', etc.) there are
-three specifications: one of how OPs should appear in the basic or exec
-modes, one of how 'goto' lines should appear (these occur in the exec
-mode only), and one of how nodes should appear in tree mode. Each has the
-same format, described below. Any text that doesn't match a special
-pattern is copied verbatim.
+For each line-style ('concise', 'terse', 'linenoise', etc.) there are
+3 format-specs which control how OPs are rendered.
+
+The first is the 'default' format, which is used in both basic and exec
+modes to print all opcodes.  The 2nd, goto-format, is used in exec
+mode when branches are encountered.  They're not real opcodes, and are
+inserted to look like a closing curly brace.  The tree-format is tree
+specific.
+
+When a line is rendered, the correct format-spec is copied and scanned
+for the following items; data is substituted in, and other
+manipulations like basic indenting are done, for each opcode rendered.
+
+There are 3 kinds of items that may be populated; special patterns,
+#vars, and literal text, which is copied verbatim.  (Yes, it's a set
+of s///g steps.)
+
+=head2 Special Patterns
+
+These items are the primitives used to perform indenting, and to
+select text from amongst alternatives.
 
 =over 4
 
@@ -982,33 +1205,64 @@ If the value of I<var> is true (not empty or zero), generates the
 value of I<var> surrounded by I<text1> and I<Text2>, otherwise
 nothing.
 
+=item B<~>
+
+Any number of tildes and surrounding whitespace will be collapsed to
+a single space.
+
+=back
+
+=head2 # Variables
+
+These #vars represent opcode properties that you may want as part of
+your rendering.  The '#' is intended as a private sigil; a #var's
+value is interpolated into the style-line, much like "read $this".
+
+These vars take 3 forms:
+
+=over 4
+
 =item B<#>I<var>
 
-Generates the value of the variable I<var>.
+A property named 'var' is assumed to exist for the opcodes, and is
+interpolated into the rendering.
 
 =item B<#>I<var>I<N>
 
-Generates the value of I<var>, left jutified to fill I<N> spaces.
+Generates the value of I<var>, left justified to fill I<N> spaces.
+Note that this means while you can have properties 'foo' and 'foo2',
+you cannot render 'foo2', but you could with 'foo2a'.  You would be
+wise not to rely on this behavior going forward ;-)
 
-=item B<~>
+=item B<#>I<Var>
 
-Any number of tildes and surrounding whitespace will be collapsed to
-a single space.
+This ucfirst form of #var generates a tag-value form of itself for
+display; it converts '#Var' into a 'Var => #var' style, which is then
+handled as described above.  (Imp-note: #Vars cannot be used for
+conditional-fills, because the => #var transform is done after the check
+for #Var's value).
 
 =back
 
-The following variables are recognized:
+The following variables are 'defined' by B::Concise; when they are
+used in a style, their respective values are plugged into the
+rendering of each opcode.
+
+Only some of these are used by the standard styles, the others are
+provided for you to delve into optree mechanics, should you wish to
+add a new style (see L</add_style> below) that uses them.  You can
+also add new ones using L<add_callback>.
 
 =over 4
 
 =item B<#addr>
 
-The address of the OP, in hexidecimal.
+The address of the OP, in hexadecimal.
 
 =item B<#arg>
 
 The OP-specific information of the OP (such as the SV for an SVOP, the
-non-local exit pointers for a LOOP, etc.) enclosed in paretheses.
+non-local exit pointers for a LOOP, etc.) enclosed in parentheses.
 
 =item B<#class>
 
@@ -1134,65 +1388,46 @@ The numeric value of the OP's type, in decimal.
 
 =back
 
-=head1 ABBREVIATIONS
-
-=head2 OP flags abbreviations
-
-    v      OPf_WANT_VOID    Want nothing (void context)
-    s      OPf_WANT_SCALAR  Want single value (scalar context)
-    l      OPf_WANT_LIST    Want list of any length (list context)
-    K      OPf_KIDS         There is a firstborn child.
-    P      OPf_PARENS       This operator was parenthesized.
-                             (Or block needs explicit scope entry.)
-    R      OPf_REF          Certified reference.
-                             (Return container, not containee).
-    M      OPf_MOD          Will modify (lvalue).
-    S      OPf_STACKED      Some arg is arriving on the stack.
-    *      OPf_SPECIAL      Do something weird for this op (see op.h)
-
-=head2 OP class abbreviations
-
-    0      OP (aka BASEOP)  An OP with no children
-    1      UNOP             An OP with one child
-    2      BINOP            An OP with two children
-    |      LOGOP            A control branch OP
-    @      LISTOP           An OP that could have lots of children
-    /      PMOP             An OP with a regular expression
-    $      SVOP             An OP with an SV
-    "      PVOP             An OP with a string
-    {      LOOP             An OP that holds pointers for a loop
-    ;      COP              An OP that marks the start of a statement
-    #      PADOP            An OP with a GV on the pad
-
 =head1 Using B::Concise outside of the O framework
 
-You can use B<B::Concise>, and call compile() directly, thereby
-avoiding the compile-only operation of O.  For example, you could use
-the debugger to step through B::Concise::compile() itself.
+The common (and original) usage of B::Concise was for command-line
+renderings of simple code, as given in EXAMPLE.  But you can also use
+B<B::Concise> from your code, and call compile() directly, and
+repeatedly.  By doing so, you can avoid the compile-time only
+operation of O.pm, and even use the debugger to step through
+B::Concise::compile() itself.
 
-When doing so, you can alter Concise output by providing new output
-styles, and optionally by adding callback routines which populate new
-variables that may be rendered as part of those styles.  For all
-following sections, please review L</FORMATTING SPECIFICATIONS>.
+Once you're doing this, you may alter Concise output by adding new
+rendering styles, and by optionally adding callback routines which
+populate new variables, if such were referenced from those (just
+added) styles.  
 
-=head2 example: Altering Concise Output
+=head2 Example: Altering Concise Renderings
 
     use B::Concise qw(set_style add_callback);
-    set_style($your_format, $your_gotofmt, $your_treefmt);
+    add_style($yourStyleName => $defaultfmt, $gotofmt, $treefmt);
     add_callback
       ( sub {
             my ($h, $op, $format, $level, $stylename) = @_;
             $h->{variable} = some_func($op);
-        }
-      );
-    B::Concise::compile(@options)->();
+        });
+    $walker = B::Concise::compile(@options,@subnames,@subrefs);
+    $walker->();
 
 =head2 set_style()
 
-B<set_style> accepts 3 arguments, and updates the three components of an
-output style (basic-exec, goto, tree). It has one minor drawback though:
-it doesn't register the style under a new name, thus you may prefer to use
-add_style() and/or set_style_standard() instead.
+B<set_style> accepts 3 arguments, and updates the three format-specs
+comprising a line-style (basic-exec, goto, tree).  It has one minor
+drawback though; it doesn't register the style under a new name.  This
+can become an issue if you render more than once and switch styles.
+Thus you may prefer to use add_style() and/or set_style_standard()
+instead.
+
+=head2 set_style_standard($name)
+
+This restores one of the standard line-styles: C<terse>, C<concise>,
+C<linenoise>, C<debug>, C<env>, into effect.  It also accepts style
+names previously defined with add_style().
 
 =head2 add_style()
 
@@ -1201,12 +1436,6 @@ above, and creates, registers, and selects the newly named style.  It is
 an error to re-add a style; call set_style_standard() to switch between
 several styles.
 
-=head2 set_style_standard($name)
-
-This restores one of the standard styles: C<terse>, C<concise>,
-C<linenoise>, C<debug>, C<env>, into effect.  It also accepts style
-names previously defined with add_style().
-
 =head2 add_callback()
 
 If your newly minted styles refer to any #variables, you'll need to
@@ -1229,29 +1458,42 @@ existing values if you need to.  The level and format are passed in as
 references to scalars, but it is unlikely that they will need to be
 changed or even used.
 
-=head2 running B::Concise::compile()
+=head2 Running B::Concise::compile()
 
 B<compile> accepts options as described above in L</OPTIONS>, and
 arguments, which are either coderefs, or subroutine names.
 
-compile() constructs and returns a coderef, which when invoked, scans
-the optree, and prints the results to STDOUT.  Once you have the
-coderef, you may change the output style; thereafter the coderef renders
-in the new style.
+It constructs and returns a $treewalker coderef, which when invoked,
+traverses, or walks, and renders the optrees of the given arguments to
+STDOUT.  You can reuse this, and can change the rendering style used
+each time; thereafter the coderef renders in the new style.
 
 B<walk_output> lets you change the print destination from STDOUT to
-another open filehandle, or into a string passed as a ref.
+another open filehandle, or (unless you've built with -Uuseperlio)
+into a string passed as a ref.
 
+    my $walker = B::Concise::compile('-terse','aFuncName', \&aSubRef);  # 1
     walk_output(\my $buf);
-    B::Concise::compile('-concise','funcName', \&aSubRef)->();
-    print "Concise Results: $buf\n";
-
-For each subroutine visited, the opcode info is preceded by a single
-line containing either the subroutine name or the stringified coderef.
-
-To switch back to one of the standard styles like C<concise> or
-C<terse>, call C<set_style_standard>, or pass the style name into
-B::Concise::compile() (as done above).
+    $walker->();                       # 1 renders -terse
+    set_style_standard('concise');     # 2
+    $walker->();                       # 2 renders -concise
+    $walker->(@new);                   # 3 renders whatever
+    print "3 different renderings: terse, concise, and @new: $buf\n";
+
+When $walker is called, it traverses the subroutines supplied when it
+was created, and renders them using the current style.  You can change
+the style afterwards in several different ways:
+
+  1. call C<compile>, altering style or mode/order
+  2. call C<set_style_standard>
+  3. call $walker, passing @new options
+
+Passing new options to the $walker is the easiest way to change
+amongst any pre-defined styles (the ones you add are automatically
+recognized as options), and is the only way to alter rendering order
+without calling compile again.  Note however that rendering state is
+still shared amongst multiple $walker objects, so they must still be
+used in a coordinated manner.
 
 =head2 B::Concise::reset_sequence()
 
@@ -1269,9 +1511,8 @@ All detected errors, (invalid arguments, internal errors, etc.) are
 resolved with a die($message). Use an eval if you wish to catch these
 errors and continue processing.
 
-In particular, B<compile> will die as follows if you've asked for a
-non-existent function-name, a non-existent coderef, or a non-CODE
-reference.
+In particular, B<compile> will die if you've asked for a non-existent
+function-name, a non-existent coderef, or a non-CODE reference.
 
 =head1 AUTHOR