832750 - Improve python 3 compat in js/src/tests/

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v1

Particularly, print as a function and the new except syntax. Also removed some unused imports as reported by pyflakes.

Comment 1

[Terrence Cole [:terrence]]

Comment on attachment 704325 [details] [diff] [review]
Patch v1

Review of attachment 704325 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for doing this!

::: js/src/tests/lib/manifest.py
@@ +170,5 @@
>          elif parts[pos] == 'pref(javascript.options.xml.content,true)':
>              testcase.options += ['-e', 'options("allow_xml");']
>              pos += 1
>          else:
> +            print('warning: invalid manifest line element "%s"'%parts[pos])

Please switch this to .format().

::: js/src/tests/lib/results.py
@@ +126,5 @@
>                  self.pb.beginline()
>  
>              if show:
>                  if self.options.show_output:
> +                    print('## %s: rc = %d, run time = %f' % (output.test.path, output.rc, output.dt), file=self.fp)

Ditto.

@@ +198,2 @@
>              for path in paths:
> +                print('    %s' % path)

Just use |print('    ' + path)| here: it's faster than entering the string formatter.

::: js/src/tests/lib/tasks_win.py
@@ +70,5 @@
>      def log(self, msg):
>          if self.verbose:
>              dd = datetime.now() - self.t0
>              dt = dd.seconds + 1e-6 * dd.microseconds
> +            print('[W%d %.3f] %s' % (self.id, dt, msg))

Switch to format.

::: js/src/tests/parsemark.py
@@ +80,5 @@
>      sigma = stddev(milliseconds, mean)
>      if not stfu:
> +        print('Runs:', [int(ms) for ms in milliseconds])
> +        print('Mean:', mean)
> +        print('Stddev: %.2f (%.2f%% of mean)' % (sigma, sigma / mean * 100))

I didn't even know about parsemark. You don't need to bother with reworking all of these prints to use .format(), if you don't want: they are more complicated and lower value.

Comment 2

[Chris Peterson [:cpeterson]]

Ms2ger: did you forget to land this r+'d patch from January 2013? :)

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Seems like it's green now. I'll land later.

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://hg.mozilla.org/mozilla-central/rev/b987f3903eb9