changes to fix Lazarus and FreePascal compilation issues.#413
changes to fix Lazarus and FreePascal compilation issues.#413pyscripter merged 3 commits intopyscripter:masterfrom
Conversation
|
current git release wasn't compiling under Lazarus/fpc, so here is the fix. |
|
|
|
now it builds, i guess you did not test it with fpc for a long time, and it did not build without the fixes in this commit. i hope i did everything right. feel free to edit my code. |
|
I see the issue. I just suggested some simplifications. |
Yes, please suggest. Or you can apply the pull request and then apply your own improvements. |
maybe I did not notice, where did you suggest some changes? |
|
i guess you were doing code review and forgot to press 'submit review' button. |
Source/PythonEngine.pas
Outdated
| NewSize := Utf8ToUnicode(PUnicodeChar(Result), Cardinal(Size + 1), Buffer, Cardinal(Size)); | ||
| {$ELSE} | ||
| NewSize := Utf8ToUnicode(PChar(Result), Cardinal(Size + 1), Buffer, Cardinal(Size)); | ||
| {$ENDIF} |
There was a problem hiding this comment.
NewSize := Utf8ToUnicode(PWideChar(Result), Cardinal(Size + 1), Buffer, Cardinal(Size));
Would work both in Delphi and Fpc.
PUnicodeChar is an alias to PWideChar;
Source/WrapDelphiClasses.pas
Outdated
| {$ENDIF} | ||
| {$IFDEF FPC} | ||
| DelphiObject := TResourceStreamClass(DelphiObjectClass).Create(LHandle, String(LResName), PChar(String(LResType))) | ||
| {$ELSE} |
There was a problem hiding this comment.
DelphiObject := TResourceStreamClass(DelphiObjectClass).Create(LHandle, String(LResName), PChar(String(LResType)))
Should work in both fpc and Delphi.
Source/WrapDelphiClasses.pas
Outdated
| {$ENDIF} | ||
| {$IFDEF FPC} | ||
| DelphiObject := TResourceStreamClass(DelphiObjectClass).CreateFromID(LHandle, LResId, PChar(String(LResType))); | ||
| {$ELSE} |
Source/WrapDelphi.pas
Outdated
| PyType: PPyTypeObject; | ||
| {$IFDEF FPC} | ||
| i: longint; | ||
| {$ENDIF} |
There was a problem hiding this comment.
i: Integer
should be in the {$ELSE} part of the the IFDEF below.
There was a problem hiding this comment.
I believe I addressed all the comments.
Thank you for those. You can take a look at another commit.
There was a problem hiding this comment.
I meant in the ELSE part of EXTENDED_RTTI! You don't need IFDEF FPC. And use Integer instead of longint.
There was a problem hiding this comment.
oh, now I see.
Thank you for comments and your patience.
Now I did it, but PR contains 3 commits.
Do you want me to figure the way to change it for 1 commit?
There was a problem hiding this comment.
so, do you like the current state of it?
would you merge? or should i change something else?
Indeed. |
|
hey, sorry for pinging you again. but would you like to merge it? or did you solve it yourself already? |
|
Sorry for the delay. Very busy right now. I will deal with PR in a couple of weeks. |
2 participants
No description provided.