Skip to content
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

Fix getImplmentation Methods #52

Open
taljacob2 opened this issue Jan 14, 2023 · 0 comments
Open

Fix getImplmentation Methods #52

taljacob2 opened this issue Jan 14, 2023 · 0 comments
Assignees
@taljacob2
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request
Milestone
Create A New `Obj...

Comments

@taljacob2
Copy link
Owner

In Object.c
Fix getImplementation Methods:

Do not EVER use concat and then free the returned conatted string.
This will cause the memberName to be freed from memory, and then you would not be able to search for the member within the Object.

Instead of using concat, you should use a const string as a memberName.
For example:

  • a "literal" const char * string
  • a #define macro that its value is a const string.

image
@taljacob2 taljacob2 added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Jan 14, 2023
@taljacob2 taljacob2 self-assigned this Jan 14, 2023
@taljacob2 taljacob2 moved this to Todo in Master Board Jan 14, 2023
@taljacob2 taljacob2 moved this from Todo to In Progress in Master Board Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@taljacob2 taljacob2

Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request
Projects
Master Board
Status: In Progress
Milestone
Create A New `Object` From `#define`
Development

No branches or pull requests
1 participant
@taljacob2