support for 32 bit systems?

[gharlan]

Do you want to support 32bit systems?

I tried to use artax on an older server, but it does not work.
The return value of getCurrentTime is too big for 32bit, so the return value is a negative int.

And in this line an error is thrown, because argument 2 is smaller than argument 1.

[kelunik]

I guess you're using PHP 7.3?

[gharlan]

No, 7.2.

[kelunik]

Does everything work fine with versions < 2.1?

[mstrouhal]

@kelunik Yes, it does. Problem started to occur after update do 2.1 version.
(PHP 7.2)

microtime(true) * 1000 is higher than PHP_INT_MAX, which is 2147483647

[qq15870939565]

How did you deal with this problem, I also encountered

[mstrouhal]

The only solution I have found so far is to require older version of amp:

"amphp/amp": "2.0.7"

[qq15870939565]

I am getting an error when adding a database. The database I am using is https://github.com/auraphp/Aura.SqlQuery

[trowski]

I pushed a commit to the fix-252 branch. Please try this on a 32-bit system and let me know if it fixes the problem.

[tunnela]

@trowski it fixes the problem 👍

[kelunik]

@gharlan @tunnela @qq15870939565 Please test with the latest master.

[qq15870939565]

OK。 Thank you 发送自 Windows 10 版邮件应用 发件人: Niklas Keller 发送时间: 2019年6月1日 2:17 收件人: amphp/amp 抄送: qq15870939565; Mention 主题: Re: [amphp/amp] support for 32 bit systems? (#252) @gharlan @tunnela @qq15870939565 Please test with the latest master. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[kelunik]

Released a bugfix version, please report if there are further issues.