Skip to content

TRTF.Create overload that takes a TEncoding parameter doesn't use it #83

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
delphidabbler opened this issue Dec 23, 2022 · 1 comment
Closed

TRTF.Create overload that takes a TEncoding parameter doesn't use it #83

delphidabbler opened this issue Dec 23, 2022 · 1 comment
Assignees
@delphidabbler
Labels
bug Bug report completed Work has been completed on this issue and changes have been committed to `develop` branch..

Comments

@delphidabbler
Copy link
Owner

codesnip/Src/URTFUtils.pas

Line 294 in 02656b8

constructor TRTF.Create(const AStream: TStream; const AEncoding: TEncoding;

AEncoding parameter never used.
@delphidabbler delphidabbler self-assigned this Dec 23, 2022
@delphidabbler delphidabbler added bug Bug report needs investigation Investigation required to confirm problem considering Issue under consideration. May be accepted or rejected labels Dec 23, 2022
@delphidabbler delphidabbler added accepted Accepted for implementation / fixing and removed needs investigation Investigation required to confirm problem considering Issue under consideration. May be accepted or rejected labels Dec 28, 2022
@delphidabbler delphidabbler added completed Work has been completed on this issue and changes have been committed to `develop` branch.. and removed accepted Accepted for implementation / fixing labels Apr 8, 2023
@delphidabbler delphidabbler added this to the Confirmed for next patch milestone Apr 8, 2023
@delphidabbler
Copy link
Owner Author

Fixed at merge commit ad9156e

@delphidabbler delphidabbler removed this from the Confirmed for next patch milestone Apr 9, 2023
@delphidabbler delphidabbler moved this to Done - Awaiting Release in CodeSnip May 23, 2023
@delphidabbler delphidabbler moved this from Done - Awaiting Release to Done - released in CodeSnip May 23, 2023
@delphidabbler delphidabbler removed this from CodeSnip Jun 18, 2023
delphidabbler added a commit that referenced this issue Jul 13, 2023
@delphidabbler
Merge branch 'feature/83-trtf-ctor-param-error' into develop
fec598e 
Fixes #83
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@delphidabbler delphidabbler

Labels
bug Bug report completed Work has been completed on this issue and changes have been committed to `develop` branch..
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@delphidabbler