Thread (9 messages) flat view 9 messages, 4 authors, 2016-06-15

Re: [PATCH] send-email: allow use of basic email list in --cc --to and --bcc

From: benoît person <hidden>
Date: 2016-06-15 22:57:48

+sub split_email_list {
+    my(@list) = @_;
+    my @tmp;
+    my @emails;
+       for (my $i = 0; $i <= $#list; $i++) {
+           if ($list[$i] =~ /,/) {
+               @emails = split(/,/, $list[$i]);
+           } else {
+               @emails = $list[$i];
+           }
+           # Removal of unwanted spaces
+           for (my $j = 0; $j <= $#emails; $j++) {
+               $emails[$j] =~ s/^\s+//;
+               $emails[$j] =~ s/\s+$//;
+           }
+           @tmp = (@tmp, @emails);
+       }
+    return(@tmp);
+}
Why two regex ? You could do something like :
$emails[$j] =~ s/^\s+|\s+$//g;
to remove leading and trailing whitespaces at the same time.

I think it's better to use the builin 'push' function to concatenate
your two arrays:
push(@tmp, @emails);

Benoit Person
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help