echo() and print() are treated as a special feature #26

New Issue

Closed

opened 2018-07-29 07:55:19 +02:00 by belliash · 2 comments

belliash commented

2018-07-29 07:55:19 +02:00

Owner

Because of some features specific to PHP (like short echo tag), PH7 Engine implemented echo() and print() builtin functions twice - as standalone function and as special construct. Thereby it is possible to call print() and echo() without brackets. In Aer, there is no such need and print() should be just a builtin function. There is also no need to maintain both functions. Thus echo() can be removed.

Because of some features specific to PHP (like short echo tag), PH7 Engine implemented echo() and print() builtin functions twice - as standalone function and as special construct. Thereby it is possible to call print() and echo() without brackets. In Aer, there is no such need and print() should be just a builtin function. There is also no need to maintain both functions. Thus echo() can be removed.

belliash added the

label 2018-07-29 07:55:19 +02:00

belliash added

and removed

labels 2018-07-29 08:20:49 +02:00

likoski commented

2018-07-29 09:04:56 +02:00

Owner

Do we want to remove echo and keep print? Or to keep echo and remove print?

Do we want to remove echo and keep print? Or to keep echo and remove print?

belliash commented

2018-07-29 19:24:43 +02:00

Poster

Owner

I was thinking about keeping print and removing echo. Do you think, this is bad idea?

I was thinking about keeping print and removing echo. Do you think, this is bad idea?

belliash changed title from ~~echo() and printf() are treated as a special feature~~ to echo() and print() are treated as a special feature

2018-07-29 20:59:03 +02:00

belliash self-assigned this 2018-07-31 10:56:18 +02:00

belliash referenced this issue from a commit

2018-08-06 17:26:50 +02:00

This commit contains several changes: * print() is now treated as standard function * get rid of echo() function * fix test suite and it fixes #26.

belliash closed this issue

2018-08-06 17:26:51 +02:00

Sign in to join this conversation.

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: aerscript/Aer#26

There is no content yet.