diff options
author | Andrew Clayton <a.clayton@nginx.com> | 2024-10-15 19:35:53 +0100 |
---|---|---|
committer | Andrew Clayton <a.clayton@nginx.com> | 2024-10-17 15:46:38 +0100 |
commit | f6036bbc7c798133e95d107ef99f289281366b0d (patch) | |
tree | 5d48299fe9b1cc5ead6aa9056915674072bb11bb /src/nxt_atomic.h | |
parent | 75b7231879bde4c9193b130a1c8b2640eda3449c (diff) | |
download | unit-f6036bbc7c798133e95d107ef99f289281366b0d.tar.gz unit-f6036bbc7c798133e95d107ef99f289281366b0d.tar.bz2 |
perl: Remove unused module constructor
In the perl language module we create a new perl *module* on the fly
comprised of some preamble, the specified perl script and some
post-amble.
In the preamble we create a constructor called new(), however this can
clash with other constructors also called new.
While this can be worked around by instead of doing
... new CLASS
rather do
... CLASS->new()
While this constructor was added in commit 3b2c1d0e ("Perl: added
implementation delayed response and streaming body."), I don't see that
we actually use it anywhere (nor is it seemingly something we document)
and if we simply remove it then things still seem to work, including the
Perl pytests
...
test/test_perl_application.py::test_perl_streaming_body_multiple_responses[5.38.2] PASSED
...
test/test_perl_application.py::test_perl_delayed_response[5.38.2] PASSED
test/test_perl_application.py::test_perl_streaming_body[5.38.2] PASSED
...
Closes: https://github.com/nginx/unit/issues/1456
Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
Diffstat (limited to 'src/nxt_atomic.h')
0 files changed, 0 insertions, 0 deletions