[elbe-devel] [PATCH 13/13] asciidoclog: enforce to log strings

[Manuel Traut]

On Tue, Dec 05, 2017 at 05:26:47PM +0100, Torben Hohn wrote:
> On Thu, Nov 30, 2017 at 03:15:15PM +0100, Manuel Traut wrote:
> 
> 
> > if not, logging might raise an exception if an integer was passed in
> > 
> > Signed-off-by: Manuel Traut <manut at linutronix.de>
> 
> Reviewed-by: Torben Hohn <torbenh at linutronix.de>

thanks for the review, applied to devel/elbe-3.0

> > ---
> >  elbepack/asciidoclog.py | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> > 
> > diff --git a/elbepack/asciidoclog.py b/elbepack/asciidoclog.py
> > index fea37212..93095a8c 100644
> > --- a/elbepack/asciidoclog.py
> > +++ b/elbepack/asciidoclog.py
> > @@ -28,7 +28,7 @@ class LogBase(object):
> >          self.fp = fp
> >  
> >      def printo(self, text=""):
> > -        self.fp.write(text+"\n")
> > +        self.fp.write("%s\n" % str(text))
> >  
> >      def print_raw(self, text):
> >          self.fp.write(text)
> > @@ -36,13 +36,13 @@ class LogBase(object):
> >      def h1(self, text):
> >          self.printo()
> >          self.printo(text)
> > -        self.printo("="*len(text))
> > +        self.printo("="*len(str(text)))
> >          self.printo()
> >  
> >      def h2(self, text):
> >          self.printo()
> >          self.printo(text)
> > -        self.printo("-"*len(text))
> > +        self.printo("-"*len(str(text)))
> >          self.printo()
> >  
> >      def table(self):
> > -- 
> > 2.15.1
> > 
> > 
> > _______________________________________________
> > elbe-devel mailing list
> > elbe-devel at linutronix.de
> > https://lists.linutronix.de/mailman/listinfo/elbe-devel
> 
> -- 
> Mit freundlichen Grüßen
> Torben Hohn
> 
> Linutronix GmbH
> 
> Standort: Bremen
> 
> Phone: +49 7556 25 999 18; Fax.: +49 7556 25 999 99
> 
> Firmensitz / Registered Office: D-88690 Uhldingen, Bahnhofstr. 3
> Registergericht / Local District Court: Amtsgericht Freiburg i. Br.; HRB
> Nr. / Trade register no.: 700 806
> 
> Geschäftsführer / Managing Directors: Heinz Egger, Thomas Gleixner
> 
> Eine Bitte von uns: Sollten Sie diese E-Mail irrtümlich erhalten haben,
> benachrichtigen Sie uns in diesem Falle bitte sobald wie es Ihnen
> möglich ist, durch Antwort-Mail. Vielen Dank!