0002919: Compile error when defining fallback implementation of apc_fetch

Viewing Issue Advanced Details [ Jump to Notes ]

[ View Simple ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0002919

[Quercus]

minor

always

09-10-08 18:54

09-11-08 14:15

Reporter

koreth

View Status

public

Assigned To

ferg

Priority

normal

Resolution

fixed

Platform

Status

closed

Projection

none

OS Version

ETA

none

Fixed in Version

3.2.1

Product Version

3.2.1

Product Build

Summary

0002919: Compile error when defining fallback implementation of apc_fetch

Description

We have code like the following in a script that can be run from the command line, where apc_fetch isn't defined:

<?php
if (!function_exists('apc_fetch')) {
function apc_fetch($key) { return false; }
}

This fails to compile:

[18:51:15.783] {resin-21} com.caucho.java.JavaCompileException: .../test__php.java:56: v_key is already defined in call(com.caucho.quercus.env.Env,com.caucho.quercus.env.Value)
[18:51:15.783] {resin-21} Value v_key = NullValue.NULL;
[18:51:15.783] {resin-21} ^
[18:51:15.783] {resin-21} 1 error

The same error happens on our similar fallback definition of apc_store(). It does not happen if I just have a parameter called "$key" in some random other function.

Low priority since I can just comment this out in our code for testing purposes.

Steps To Reproduce

Additional Information

Attached Files

Relationships

Notes
(0003420) ferg 09-11-08 11:36	I can't reproduce this. That code snippet works fine for me. Is it possible to either create a more complete test case or send the generated code?

(0003423) koreth 09-11-08 12:03	That exact test case causes the error for me. The original real-world case is of course part of a larger file but before filing the bug I tested it in isolation and got the same error. Just tried again and it still happens with a clean build of Resin/Pro. Maybe there's a fix in the internal code base you're using that hasn't been pushed out to the public svn servers yet?

(0003425) ferg 09-11-08 12:39	I just checked in the most recent changes.

(0003426) koreth 09-11-08 13:52	Updated Resin source from svn and the problem went away.

(0003428) ferg 09-11-08 14:15	php/37f8

Issue History
Date Modified	Username	Field	Change
09-10-08 18:54	koreth	New Issue
09-11-08 11:36	ferg	Note Added: 0003420
09-11-08 12:03	koreth	Note Added: 0003423
09-11-08 12:39	ferg	Note Added: 0003425
09-11-08 13:52	koreth	Note Added: 0003426
09-11-08 14:15	ferg	Note Added: 0003428
09-11-08 14:15	ferg	Assigned To	=> ferg
09-11-08 14:15	ferg	Status	new => closed
09-11-08 14:15	ferg	Resolution	open => fixed
09-11-08 14:15	ferg	Fixed in Version	=> 3.2.1

Mantis 1.0.0rc3[^]

37 total queries executed.
30 unique queries executed.