Skip to content

A few small examples I've been meaning to clean up - #29

Merged
septract merged 4 commits into
mainfrom
mdd/residual-simple-examples
Jul 4, 2024
Merged

septract merged 4 commits into
mainfrom
mdd/residual-simple-examples

Conversation

@septract

Copy link
Copy Markdown
Collaborator
  • A couple of small examples of CN syntax
  • Some small error cases I found

@septract
septract requested review from cp526 and dc-mak June 13, 2024 23:05
@dc-mak

dc-mak commented Jun 17, 2024

Copy link
Copy Markdown
Collaborator

Some of these have links to issues attached and some of these don't. Is that intentional? I'm fine to let it be like this, but just wanted to check.

@dc-mak dc-mak left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving to not delay by another day but would still like clarifications requested.

Comment on lines +3 to +4
#include "list.h"
#include "list_snoc_spec.h"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you're running within the same directory, it should be:

#include "../../../examples/list.h"
#include "../../../examples/list_snoc.h"

Alternatively, copy over the headers into here if you want?

What's the purpose of this example? Is it a bad error message or something else?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is redundant now @bcpierce00 has finished his queue example. I'll delete it.

@septract
septract merged commit 8d4eafa into main Jul 4, 2024
@septract
septract deleted the mdd/residual-simple-examples branch July 4, 2024 19:26
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

Successfully merging this pull request may close these issues.

2 participants