Issue 83: Failure to set FCurrentFile -- is this still valid?

Open EganSolo opened this issue 8 years ago • 1 comments

In the HTMLView unit, on line 1,083, there is the following statement:

  //FCurrentFile := Name; //BG, 03.04.2011: issue 83: Failure to set FCurrentFile

Is this still a valid bug? Can we uncoment that line now?

FYI: I've uncommented this line, recompiled and as far as my use-case goes which is to display a complicated html file, nothing blew up.

Oct 13 '17 17:10 EganSolo

We can remove the line as FCurrentFile is set later in LoadDocument in HtmlView.pas line 1283. Do you have any issues without setting FCurrentLine in LoadFile()?

Oct 22 '17 10:10 BerndGabriel