search

pavkam / tzdb

Delphi/FPC Time Zone Database
https://www.iana.org/time-zones
BSD 3-Clause "New" or "Revised" License
84 stars 27 forks source link

Please use fully qualified unitname #31

Closed SteveNew closed 4 years ago

SteveNew commented 4 years ago

Any chance that in TZDB.pas all Delphi units are fully qualified - SysUtils -> System.SysUtils?

There are only 17 places :) to make the file "more" correct - for us people who do not want any aliases.

Thanks - and great work.

pavkam commented 4 years ago

Hello,

Can you provide a Pull Request with these changes? I do not have Delphi, only FPC. I add aliases for Delphi but I can't test.

Alex.

SteveNew commented 4 years ago

Hi Alex. I do not know if you want to invest time in it, but I guess using the Community Edition of Delphi these days to test - would be fine. I hope the PR can be accepted an does not break any FPC things - since I do not use FPC :)

pavkam commented 4 years ago

Thanks Steve,

I will give it a try this weekend.

Alex.

On Mon, Jun 22, 2020 at 8:57 AM SteveNew notifications@github.com wrote:

Hi Alex. I do not know if you want to invest time in it, but I guess using the Community Edition of Delphi these days to test - would be fine. I hope the PR can be accepted an does not break any FPC things - since I do not use FPC :)

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/pavkam/tzdb/issues/31#issuecomment-647352379, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABX44TK7FOACKZW4ZYB4PYLRX4FIFANCNFSM4OCSA2WA .

-- Alexandru Ciobanu Web: https://alex.ciobanu.org/ http://alex.ciobanu.org/ Email: alex@ciobanu.org