Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor(net:message) add NetMessage interface #62

Merged
merged 2 commits into from
Sep 13, 2014
Merged

Conversation

btc
Copy link
Contributor

@btc btc commented Sep 13, 2014

  • Design Goal: reduce coupling
  • NB: Slices hold references to an underlying array, and if you assign
    one slice to another, both refer to the same array. If a function
    takes a slice argument, changes it makes to the elements of the slice
    will be visible to the caller, analogous to passing a pointer to the
    underlying array.

* Design Goal: reduce coupling
* NB: Slices hold references to an underlying array, and if you assign
  one slice to another, both refer to the same array. If a function
  takes a slice argument, changes it makes to the elements of the slice
  will be visible to the caller, analogous to passing a pointer to the
  underlying array.
@btc btc added the status/in-progress In progress label Sep 13, 2014
@btc
Copy link
Contributor Author

btc commented Sep 13, 2014

@jbenet

@btc btc added codereview and removed status/in-progress In progress labels Sep 13, 2014
@btc btc self-assigned this Sep 13, 2014
@jbenet
Copy link
Member

jbenet commented Sep 13, 2014

LGTM!

btc pushed a commit that referenced this pull request Sep 13, 2014
refactor(net:message) add NetMessage interface
@btc btc merged commit bb58203 into net Sep 13, 2014
@btc btc removed the codereview label Sep 13, 2014
@btc btc deleted the net-msg-interface branch September 13, 2014 03:25
@btc btc removed their assignment Mar 30, 2015
@aschmahmann aschmahmann mentioned this pull request Sep 22, 2020
72 tasks
ribasushi pushed a commit that referenced this pull request Jul 4, 2021
Deprecate EventLogging and Warning* functions
ariescodescream pushed a commit to ariescodescream/go-ipfs that referenced this pull request Oct 23, 2021
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Feb 25, 2022
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Feb 25, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants