René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Wrong conversion of division to old_div","articleBody":"Example code\r\n===========\r\nThe following code contains an old style division. \r\n```python\r\n(x / 2 * 3.0)\r\n```\r\n\r\nThe expected result\r\n================\r\nThe result running the above code with arbitrary values assigned to x should be the following, where the parentheses explicitly show the order of the operations `(x / 2) * 3.0`. Or written in another order the result should be evaluated as `x * (3.0 / 2)`.\r\n\r\nThe issue\r\n========\r\nOnce we run `futurize` on the code above, an `old_div` gets inserted by the `fix_division_safe` fixture. However as one can see in the diff below, the function gets called with a wrong order of arguments.\r\n```bash\r\n$ futurize --stage2 src/example.py\r\nRefactoringTool: Refactored src/example.py\r\n--- src/example.py      (original)\r\n+++ src/example.py      (refactored)\r\n@@ -1 +1,3 @@\r\n-(x / 2 * 3.0)\r\n+from __future__ import division\r\n+from past.utils import old_div\r\n+(old_div(x, 2 * 3.0))\r\nRefactoringTool: Files that need to be modified:\r\nRefactoringTool: src/example.py\r\n```\r\n\r\nExpected conversion\r\n=================\r\nAs already stated in the section 'expected result' the correct conversion should have been: `(old_div(x, 2) * 3.0)`","author":{"url":"https://github.com/wagnerpeer","@type":"Person","name":"wagnerpeer"},"datePublished":"2019-01-30T19:08:31.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":12},"url":"https://github.com/434/python-future/issues/434"}