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.
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
[Feature request] Formalize/Make-explicit the data structures used throughout org-fc #94
Comments
I agree that this would help with the readability of the code. A few years ago I thought about switching the card types to use eieio, I can't remember why I decided against it at that time, here it seems like it could lead to a number of invasive changes, exchanging the learning curve of figuring out which element of a list is which to the learning curve of understanding the details of eieio.
|
Looks like In this case my argument about the added learning curve is invalid, |
I had a hard time following the code the first time I read through because the representation of a card/position/etc isn't immediately clear. I expected to find something like
org-fc-review-session
or acl-defstruct
for each of the nouns used inorg-fc
(e.g. position, card, index). How do you feel about adding moredefclass
definitions (e.g.org-fc-position
).Based on what I can tell, those structures could look like:
`org-fc-card`
`org-fc-position`
`org-fc-review-session-rating`
The text was updated successfully, but these errors were encountered: