Topics

Attendance

• Jane Brandenburg - PHS
• Allen Brown - Scalawags
• Ray Cochran - PHS
• Gary Oliver - Scalawags
• Jeff Rice - Scalawags
• Asumu Takikawa - CV

Current Documentation

• ElectricalPower20041107.html -- Electrical Power connections
• ElectricalNodes20041107.html -- Electrical Communication Links
• CvsInstructions20041111.html -- How to use CVS -- Updated
• ProtocolCommandTable20041007.html -- Packet and Commands: Tabular definition
• ProtocolCommandState20041107.html -- Packet and Commands: BNF and State Diagrams
• DirectoryStructure20041022.html -- How the project is organized on disk
• Pictures/CallSequence20041114.gif -- Heirarchy of calls from packet on down. -- New

Action Items from last week

• Finish cn_data() -- Jeff - Not done.
• Fix Packet Definition drawing so every transition has one condition -- Jeff - Not done.
• Write cn_address(), cn_time(), cn_length() -- Jeff - Reassigned.
• Write cn_error() -- Sean
• Recruit John (of PHS) and possibly his laptop -- Ray - not done
• Review C -- Peter - not done
• Fix script to copy template to new function name. -- Allen - done

Changes

The CvsInstructions link (above) has been updated. It now has separate instructions for when you are logged directly into scalawags.org.

I've noticed there are problems with information not being available in the right places. In particular, src was not provided to cn_command(). Gary suggested we pass the info in a struct. The struct will absorb the current PayloadPointer.

But I'm uncomfortable making a change like this without seeing all of the current functions compile. So I will hold off with the rollout.

I noticed several functions #defining the same thing, with different names. Specifically, the start node number and the end node number. I will check in a new file: src/cn_defines.h that will define these. Please review this file and change your files to work with it, removing your own defines.

The make now depends on the library desktopsim. But the Makefile doesn't yet understand that dependency. So you have to explicitly make it first. Unfortunately this wont work until all the library functions at least compile.

  make desktopsim

Action Items before next week

• Finish cn_data() -- Jeff
• Asumu has a bunch more assigned functions. See assignments below.
• Fix cn_charClassifier() warning -- Sean
• Fix Packet Definition drawing so every transition has one condition -- Jeff
• Recruit John (of PHS) and possibly his laptop -- Ray
• Review C -- Peter

Schedule

• Nov 19 - location: John's house
  • Assemble the functions into a simple test program.
  • Map the high level simulation code structure.
  • Assign the next level of coding.
  • Code Review/Workshop.
• Nov 26 - location: John's house Note: Allen won't be there
  • Code Review/Workshop.
• Dec 3 - location: John's house
  • Code Review/Workshop.
• Dec 10 - location: John's house
  • Code Review/Workshop.
• Dec 17 - location: John's house
  • Code Review/Workshop.

Function Status

• cn_address - good
• cn_charClassifier - warning remains
  • ../src/cn_charClassifier.c:25: warning: deprecated use of label at end of compound statement
• cn_command - good
• cn_commands - The test is missing. Warnings Because of the complexity of cn_command, I have reinstated this.
  • ../src/cn_commands.c: In function `commands':
  • ../src/cn_commands.c:21: warning: assignment makes pointer from integer without a cast
  • ../src/cn_commands.c:24: warning: implicit declaration of function `command'
  • ../src/cn_commands.c:24: warning: assignment makes pointer from integer without a cast
• cn_configrd - good
• cn_configwr - good
• cn_configis - needs to be written. not yet assigned
• cn_data - Warning. What is atoi doing here still?
  • ../src/cn_data.c:12: warning: implicit declaration of function `atoi'
• cn_error - good
• cn_read - good
• cn_self_address - no test written
• cn_write - good
• cn_htoi - no test written.
• cn_length - fails to compile
  • cn_length_t.c:68: conflicting types for `cn_error'
  • ../src/cn_error.h:4: previous declaration of `cn_error'
  • ../src/cn_length.c: In function `cn_length':
  • ../src/cn_length.c:36: too few arguments to function `cn_error'
• cn_level - needs to be written
• cn_time - needs to be written

Assignments

• int cn_data(char *packetPayload) -- Asumu
• int cn_datais(int src, addr, data) -- Asumu
• int cn_time(char *packetPayload) -- Asumu
• int cn_length(char *packetPayload) -- Asumu
• int cn_level(char *packetPayload) -- Asumu
• enum cn_charClassifier(char) -- Sean
• cn_configIs(int src, addr, data) -- Asumu
• cn_periodic(int src, addr, time, length, char *packetPayload) -- Asumu
• call sequence graph -- Allen