[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[6233] reduce use of method call syntax
From: |
Gavin D. Smith |
Subject: |
[6233] reduce use of method call syntax |
Date: |
Thu, 30 Apr 2015 18:58:03 +0000 |
Revision: 6233
http://svn.sv.gnu.org/viewvc/?view=rev&root=texinfo&revision=6233
Author: gavin
Date: 2015-04-30 18:58:01 +0000 (Thu, 30 Apr 2015)
Log Message:
-----------
reduce use of method call syntax
Modified Paths:
--------------
trunk/tp/Texinfo/Convert/Plaintext.pm
Modified: trunk/tp/Texinfo/Convert/Plaintext.pm
===================================================================
--- trunk/tp/Texinfo/Convert/Plaintext.pm 2015-04-30 18:20:56 UTC (rev
6232)
+++ trunk/tp/Texinfo/Convert/Plaintext.pm 2015-04-30 18:58:01 UTC (rev
6233)
@@ -717,7 +717,9 @@
{
my $self = shift;
my $text = shift;
- $self->{'count_context'}->[-1]->{'bytes'} += $self->count_bytes($text);
+ $self->{'count_context'}->[-1]->{'bytes'}
+ += Texinfo::Common::count_bytes($self, $text,
+ $self->{'output_perl_encoding'});
}
sub _add_lines_count($$)
@@ -793,8 +795,8 @@
my $self = shift;
if (defined($self->{'empty_lines_count'})
and $self->{'empty_lines_count'} == 0) {
- $self->_add_text_count("\n");
- $self->_add_lines_count(1);
+ _add_text_count($self, "\n");
+ _add_lines_count($self, 1);
$self->{'empty_lines_count'} = 1;
return "\n";
}
@@ -811,14 +813,14 @@
my $result = '';
if (scalar(@{$self->{'pending_footnotes'}})) {
- $result .= $self->_add_newline_if_needed();
+ $result .= _add_newline_if_needed($self);
print STDERR "FOOTNOTES ".scalar(@{$self->{'pending_footnotes'}})."\n"
if ($self->{'debug'});
if ($self->get_conf('footnotestyle') eq 'end' or !defined($element)) {
my $footnotes_header = " ---------- Footnotes ----------\n\n";
$result .= $footnotes_header;
- $self->_add_text_count($footnotes_header);
- $self->_add_lines_count(2);
+ _add_text_count($self, $footnotes_header);
+ _add_lines_count($self, 2);
$self->{'empty_lines_count'} = 1;
} else {
@@ -868,11 +870,11 @@
$result .= $footnote_text;
$self->{'text_element_context'}->[-1]->{'counter'} +=
Texinfo::Convert::Unicode::string_width($footnote_text);
- $self->_add_text_count($footnote_text);
+ _add_text_count($self, $footnote_text);
$self->{'empty_lines_count'} = 0;
$result .= $self->_convert($footnote->{'root'}->{'args'}->[0]);
- $result .= $self->_add_newline_if_needed();
+ $result .= _add_newline_if_needed($self);
my $old_context = pop @{$self->{'context'}};
die if ($old_context ne 'footnote');
@@ -1135,8 +1137,8 @@
if ($menu_command->{'cmdname'} eq 'menu') {
my $result = "* Menu:\n\n";
- $self->_add_text_count($result);
- $self->_add_lines_count(2);
+ _add_text_count($self, $result);
+ _add_lines_count($self, 2);
return $result;
} else {
return '';
@@ -1210,12 +1212,12 @@
}
my $result = '';
- $result .= $self->_add_newline_if_needed();
+ $result .= _add_newline_if_needed($self);
if ($in_info) {
my $info_printindex_magic = "\x{00}\x{08}[index\x{00}\x{08}]\n";
$result .= $info_printindex_magic;
- $self->_add_text_count($info_printindex_magic);
- $self->_add_lines_count(1);
+ _add_text_count($self, $info_printindex_magic);
+ _add_lines_count($self, 1);
}
my $heading = "* Menu:\n\n";
@@ -1293,18 +1295,18 @@
} else {
$entry_counts{$entry_text}++;
$entry_nr = ' <'.$entry_counts{$entry_text}.'>';
- $self->_add_text_count($entry_nr);
+ _add_text_count($self, $entry_nr);
}
my $entry_line = "* $entry_text${entry_nr}: ";
- $self->_add_text_count("* ".": ");
- #$self->_add_text_count($entry_line);
+ _add_text_count($self, "* ".": ");
+ #_add_text_count($self, $entry_line);
my $line_width = Texinfo::Convert::Unicode::string_width($entry_line);
my $entry_line_addition = '';
if ($line_width < $index_length_to_node) {
my $spaces = ' ' x ($index_length_to_node - $line_width);
$entry_line_addition .= $spaces;
- $self->_add_text_count($spaces);
+ _add_text_count($self, $spaces);
}
my $node = $entry_nodes{$entry};
@@ -1337,7 +1339,7 @@
$self->{'count_context'}->[-1]->{'bytes'} += $byte_count;
}
$entry_line_addition .= '.';
- $self->_add_text_count('.');
+ _add_text_count($self, '.');
$entry_line .= $entry_line_addition;
$result .= $entry_line;
@@ -1356,14 +1358,14 @@
= ' ' x ($self->{'fillcolumn'} - $line_part_width - $line_width)
. "$line_part\n";
}
- $self->_add_lines_count(1);
- $self->_add_text_count($line_part);
+ _add_lines_count($self, 1);
+ _add_text_count($self, $line_part);
$result .= $line_part;
}
$result .= "\n";
- $self->_add_text_count("\n");
- $self->_add_lines_count(1);
+ _add_text_count($self, "\n");
+ _add_lines_count($self, 1);
return $result;
}
@@ -1409,8 +1411,8 @@
$self->{'count_context'}->[-1]->{'lines'} -= 1;
}
$text .= "\n";
- $self->_add_text_count("\n");
- $self->_add_lines_count(1);
+ _add_text_count($self, "\n");
+ _add_lines_count($self, 1);
return $text;
}
@@ -1572,7 +1574,7 @@
if ($root->{'text'} =~ /\f/) {
$result = _get_form_feeds($root->{'text'});
}
- $self->_add_text_count($result);
+ _add_text_count($self, $result);
print STDERR "IGNORABLE SPACE: ($result)\n" if ($self->{'debug'});
return $result;
}
@@ -1594,7 +1596,7 @@
$result = "";
if ($root->{'text'} =~ /\f/) {
$result .= _get_form_feeds($root->{'text'});
- $self->_add_text_count($result);
+ _add_text_count($self, $result);
}
$result .= _count_added($self, $formatter->{'container'},
$formatter->{'container'}->add_text("\n"));
@@ -1621,7 +1623,7 @@
return $result;
# the following is only possible if paragraphindent is set to asis
} elsif ($root->{'type'} and $root->{'type'} eq
'empty_spaces_before_paragraph') {
- $self->_add_text_count($root->{'text'});
+ _add_text_count($self, $root->{'text'});
return $root->{'text'};
# ignore text outside of any format, but warn if ignored text not empty
} elsif ($root->{'text'} =~ /\S/) {
@@ -1907,8 +1909,8 @@
$result = _count_added($self, $formatter->{'container'},
$formatter->{'container'}->add_pending_word(1));
my ($image, $lines_count) = $self->_image($root);
- $self->_add_lines_count($lines_count);
- $self->_add_text_count($image);
+ _add_lines_count($self, $lines_count);
+ _add_text_count($self, $image);
if ($image ne '' and $formatter->{'type'} ne 'paragraph') {
$self->{'empty_lines_count'} = 0;
}
@@ -2226,8 +2228,8 @@
$self->get_conf('NUMBER_SECTIONS'),
($self->{'format_context'}->[-1]->{'indent_level'}) * $indent_length);
$self->{'empty_lines_count'} = 0 unless ($result eq '');
- $self->_add_text_count($result);
- $self->_add_lines_count(2);
+ _add_text_count($self, $result);
+ _add_lines_count($self, 2);
return $result;
} elsif ($command eq 'U') {
@@ -2394,7 +2396,7 @@
= $self->{'empty_lines_count'};
$self->{'document_context'}->[-1]->{'in_multitable'}++;
} elsif ($root->{'cmdname'} eq 'float') {
- $result .= $self->_add_newline_if_needed();
+ $result .= _add_newline_if_needed($self);
if ($root->{'extra'} and $root->{'extra'}->{'normalized'}) {
$result .= $self->_anchor($root);
}
@@ -2413,8 +2415,8 @@
if ($contents ne '') {
$contents .= "\n";
$self->{'empty_lines_count'} = 1;
- $self->_add_text_count($contents);
- $self->_add_lines_count($lines_count+1);
+ _add_text_count($self, $contents);
+ _add_lines_count($self, $lines_count+1);
}
$self->{'setcontentsaftertitlepage_done'} = 1;
$result .= $contents;
@@ -2428,8 +2430,8 @@
if ($contents ne '') {
$contents .= "\n";
$self->{'empty_lines_count'} = 1;
- $self->_add_text_count($contents);
- $self->_add_lines_count($lines_count+1);
+ _add_text_count($self, $contents);
+ _add_lines_count($self, $lines_count+1);
}
$self->{'setshortcontentsaftertitlepage_done'} = 1;
@@ -2462,13 +2464,13 @@
$self->get_conf('NUMBER_SECTIONS'),
($self->{'format_context'}->[-1]->{'indent_level'})
* $indent_length);
- $result .= $self->_add_newline_if_needed();
+ $result .= _add_newline_if_needed($self);
$self->{'empty_lines_count'} = 0 unless ($heading_underlined eq '');
- $self->_add_text_count($heading_underlined);
+ _add_text_count($self, $heading_underlined);
$result .= $heading_underlined;
if ($heading_underlined ne '') {
$self->_add_lines_count(2);
- $result .= $self->_add_newline_if_needed();
+ $result .= _add_newline_if_needed($self);
}
}
$self->{'format_context'}->[-1]->{'paragraph_count'} = 0;
@@ -2670,8 +2672,8 @@
pop @{$self->{'count_context'}};
}
$self->{'format_context'}->[-1]->{'paragraph_count'}++;
- $self->_add_text_count($result);
- $self->_add_lines_count($lines_count);
+ _add_text_count($self, $result);
+ _add_lines_count($self, $lines_count);
return $result;
} elsif ($root->{'cmdname'} eq 'sp') {
if ($root->{'extra'}->{'misc_args'}->[0]) {
@@ -2694,8 +2696,8 @@
($result, $lines_count)
= $self->_contents($self->{'structuring'}->{'sectioning_root'},
'contents');
- $self->_add_lines_count($lines_count);
- $self->_add_text_count($result);
+ _add_lines_count($self, $lines_count);
+ _add_text_count($self, $result);
}
return $result;
} elsif ($root->{'cmdname'} eq 'shortcontents'
@@ -2707,8 +2709,8 @@
($result, $lines_count)
= $self->_contents($self->{'structuring'}->{'sectioning_root'},
'shortcontents');
- $self->_add_lines_count($lines_count);
- $self->_add_text_count($result);
+ _add_lines_count($self, $lines_count);
+ _add_text_count($self, $result);
}
return $result;
# all the @-commands that have an information for the formatting, like
@@ -3224,7 +3226,7 @@
or defined($root->{'number'})
or $root->{'extra'}->{'caption'} or
$root->{'extra'}->{'shortcaption'})) {
- $result .= $self->_add_newline_if_needed();
+ $result .= _add_newline_if_needed($self);
my ($caption, $prepended) = Texinfo::Common::float_name_caption($self,
$root);
if ($prepended) {
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [6233] reduce use of method call syntax,
Gavin D. Smith <=