Modify ↓

Opened 5 years ago

Closed 5 years ago

#770 closed defect (fixed)

Wrong variable assignment in org.openstreetmap.josm.io.MyHttpHandler

Reported by: krasnoj@… Owned by: framm
Priority: major Component: Core
Version: Keywords: wrong variable assignment
Cc: krasnoj@…

Description

Hi!

I think the function:
public MyHttpHandler(String proxy, int port) {

proxy = proxy;
proxyPort = port;

}

is wrong. The right code should be:
public MyHttpHandler(String proxy, int port) {

this.proxy = proxy;
this.proxyPort = port;

}

If you write proxy=proxy, this assignment has no effect to the class variable proxy. Instead this.proxy=proxy overwrites the class variable (instead the function argument) like it is wantet.

Attachments (0)

Change History (1)

comment:1 Changed 5 years ago by anonymous

  • Resolution set to fixed
  • Status changed from new to closed

Closed in r655.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.