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.

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

Clarifications #11

Open
2shaar2059 opened this issue Jul 6, 2022 · 0 comments
Open

Clarifications #11

2shaar2059 opened this issue Jul 6, 2022 · 0 comments

Comments

@2shaar2059
Copy link

  1. In SCH_DefaultScheduleTable (from sch_def_schtbl.c), is there a particular reason that certain entries are commented out and have their EnableState set to SCH_DISABLED? Isn't this redundant (i.e. does this have the same result as simply uncommenting those lines and keeping their disabled status)?
  2. Can the order of entries in SCH_DefaultScheduleTable be rearranged without affecting the results of the scheduler app?
  3. Despite being commented out, is there a particular reason for putting entries with different MessageIndex out of order and spaced out by several SCH_UNUSED entries? For example, the following lines of code are seperated by many SCH_UNUSED entries and appear out of order by MessageIndex
    /*{ SCH_DISABLED, SCH_ACTIVITY_SEND_MSG, 4, 1, 6, SCH_GROUP_CFS_HK }, */ /* CS HK Request */ has a MessageIndex of 6 and appears on line 122
    /*{ SCH_DISABLED, SCH_ACTIVITY_SEND_MSG, 1, 0, 24, SCH_GROUP_MD_WAKEUP }, */ /* MD Wakeup */ has a MessageIndex of 24 and appears on line 130
    /*{ SCH_DISABLED, SCH_ACTIVITY_SEND_MSG, 4, 2, 7, SCH_GROUP_CFS_HK }, */ /* DS HK Request */ has a MessageIndex of 7 and appears on line 159

4./*{ SCH_DISABLED, SCH_ACTIVITY_SEND_MSG, 1, 0, 24, SCH_GROUP_MD_WAKEUP }, */ /* MD Wakeup */ appears 20 times in the file. Is there a reason for this?
5. Is there any significance to separating entries into slots of 5 (or slots at all) in the layout of the array? Is this simply a matter of styling or is there a concept of slots in the code too? If so, what is the point of that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant