locutusjs / locutus

Bringing stdlibs of other programming languages to JavaScript for educational purposes

Home Page:https://locutus.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

php/network/long2ip is missing the first & 0xFF

mkg20001 opened this issue · comments

  • Have you checked the guidelines in our Contributing document?

Description

In src/php/network/long2ip.js the first " & 0xFF" is missing.

-  return [ip >>> 24, ip >>> 16 & 0xFF, ip >>> 8 & 0xFF, ip & 0xFF].join('.')
+  return [ip >>> 24 & 0xFF, ip >>> 16 & 0xFF, ip >>> 8 & 0xFF, ip & 0xFF].join('.')
commented

@kvz this issue should be closed.

@argosback made you a collaborator so you can do this yourself from now on 😄