On 2024-Dec-03, Nathan Bossart wrote: > +Subprogram > +parse_subprogram(const char *name) > +{
Please add a comment atop this function. Also, I don't think it should go at the end of the file; maybe right after main() is a more appropriate location? > +/* special must-be-first options for dispatching to various subprograms */ > +typedef enum Subprogram > +{ I'm not sure this comment properly explains what this enum is used for. Maybe add a reference to parse_subprogram to the comment? Thanks -- Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/ Al principio era UNIX, y UNIX habló y dijo: "Hello world\n". No dijo "Hello New Jersey\n", ni "Hello USA\n".