Skip to content

Cache improvement#56

Closed
vortigont wants to merge 1 commit intoivanseidel:masterfrom
vortigont:cacheupd
Closed

Cache improvement#56
vortigont wants to merge 1 commit intoivanseidel:masterfrom
vortigont:cacheupd

Conversation

@vortigont
Copy link

  • cached node link updates on add/remove also
  • ListNode has a constructor now, simplifies code, ensures *next is a nullptr if unlinked
  • some code sanity checks and cleanup
  • add head()/tail()/exist() helpers to make user code simple on Node access/checks
  • unit tests updated for new helpers

 - cached node link updates on add/remove also
 - ListNode has a constructor now, simplifies code, ensures *next is a nullptr if unlinked
 - some code sanity checks and cleanup
 - add head()/tail()/exist() helpers to make user code simple on Node access/checks
 - unit tests updated for new helpers

Signed-off-by: Emil Muratov <gpm@hotplug.ru>
@ivanseidel
Copy link
Owner

Really interesting. Its missing a few docs for people to know how to use the new methods. Also, have you tested on arduino platform? is it compliant?

@vortigont
Copy link
Author

for sure it works pretty fine with Arduino. Overloaded constructors mostly used internally to optimize node adding/creating, not much of a use for user actually, unless you want to create a list of lists :))
The other methods like head()/tail() are pretty obvious, but could be mentioned in examples also.

@vortigont vortigont closed this Dec 13, 2024
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