norgepaul / TChromeTabs

Comprehensive Delphi implementation of Chrome's tab system
Other
215 stars 78 forks source link

ChromeTabsGlassForm compilation fails #26

Closed ErikVirtel closed 8 years ago

ErikVirtel commented 9 years ago

Due to the latest changes from September 3, 2014 ChromeTabsGlassForm.pas is not compilable anymore.

In Line 34 of ChromeTabsGlassForm.pas {$ENDIF} was used instead of the correct {$IFEND}

landrix commented 9 years ago

Do you use an older delphi Version?

ErikVirtel commented 9 years ago

Delphi XE2.

norgepaul commented 9 years ago

Can you try the new version and see if it compiles?

ErikVirtel commented 9 years ago

Since there is still the ENDIF statement, the behavior did not change.

But I have found some information about the issue.

http://docwiki.embarcadero.com/RADStudio/XE8/en/Legacy_IFEND_(Delphi)

At the XE4 release, the Delphi compilers were changed to accept either $IFEND or $ENDIF to close $IF statements. Before XE4, only $IFEND could be used to close $IF statements.

My point is, when you introduced these compiler switches on 2014-09-03, you used IFEND in every other file but only in ChromeTabsGlassForm ENDIF was used.

landrix commented 9 years ago

seemingly some merge problems

look at this repo

https://github.com/landrix/TChromeTabs

norgepaul commented 9 years ago

Hi Sven,

seemingly some merge problems

look at this repo

https://github.com/landrix/TChromeTabs

Yes, we merged in some changes yesterday, but they're working fine for me. Can you be more specific about the issues you're having?

/Paul

landrix commented 9 years ago

My unit ChromeTabsGlassForm;

uses Windows, Messages, SysUtils, Classes, Forms, Controls, {$IF CompilerVersion >= 28.0}System.{$ifend}Types, ChromeTabs;

your unit ChromeTabsGlassForm;

{$IF CompilerVersion >= 28.0} System.Types, {$else} Types, {$endif}

git shows no different? why?

norgepaul commented 8 years ago

Sorry, you'd need to ask GitHub support.