Rethink class hierarchy
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
- Issue type
- Refactor
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- cpp
- Domain
- operating-systems
Research direction
Start by reading the existing ILEFunction and PGMFunction classes, focusing on their resolving and loading responsibilities and how direct program or library loading currently works. Compare those responsibilities with the proposed MIObject, ILEServiceProgram, ILESymbol, and ILEFunction hierarchy. Done means the reusable loading and resolving behavior is separated without breaking the simple direct-loading case.
Written by the indexing model from the issue text.
Description
Currently ILEFunction and PGMFunction handle a lot of things like i.e. resolving or loading that could be reused across multiple instan, or split it up to better handle i.e. missing functionality without crashing.
Something like (hierarchy matches how you):
MIObject(wrap_RSLOBJ)PGMFunction(wrap_PGMCALL)ILEServiceProgram(wrap_ILELOAD), can be loaded from paths orMIObjectILESymbol(wrap_ILESYM), could be used for either code or dataILEFunction(wrap_ILECALL), could either wrapILESymbolfor code or be distinct_ILESYMcaller
You'd want to handle the easy case of loading a program/library directly with these classes still, to avoid boilerplate for the simple case.
- Dominant language
- C++
- Stars
- 3
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from SeidenGroup/pase-cpp
-
Generate from PCML Openenhancement
Difficulty 5/5 Over a week Newbie friendliness 35/100
SeidenGroup/pase-cpp#7 ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
SeidenGroup/pase-cpp#6 · 2 comments ·
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 25/100
SeidenGroup/pase-cpp#2 ·
All issues in SeidenGroup/pase-cpp
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
AXERA-TECH/ax-llm#77 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
games-on-whales/wolf#509 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
bug-unconfirmed
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
NVIDIA/cuda-samples#453 ·