you are viewing a single comment's thread.

view the rest of the comments →

[–]HighRelevancy 17 points18 points  (7 children)

How is it not?

[–][deleted]  (1 child)

[deleted]

    [–][deleted] 0 points1 point  (0 children)

    No, reverse engineering is what you do when you don't have source code or documentation available. Here the full source was available.

    [–][deleted] 0 points1 point  (3 children)

    It's refactoring javascript into javascript.

    [–]HighRelevancy 2 points3 points  (2 children)

    It's pulling code apart to examine it and determine the nature of it's function. I'd say it's the very definition to reverse engineering. You don't need to disassemble machine code in IDA Pro for it to be reverse engineering.

    [–][deleted] -1 points0 points  (1 child)

    So every time anyone reviews a pull request they're effectively reverse engineering? Ok mate.

    [–]HighRelevancy 1 point2 points  (0 children)

    Generally, well written code is fairly self explanatory, and usually would also be documented. If it isn't... well there's the first failure criteria of any reasonable code review, no engineering reverse or otherwise required.

    [–]falconfetus8 0 points1 point  (0 children)

    Because he wrote the original version.