Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Mar 12 10:08
    SuperOP535 closed #92
  • Jan 13 21:26
    rom1504 commented #95
  • Jan 13 18:12
    rom1504 opened #96
  • Jan 13 18:11
    rom1504 opened #95
  • Nov 29 2018 20:42

    rom1504 on master

    Change some functions to arrow … (compare)

  • Nov 29 2018 20:42
    rom1504 closed #94
  • Nov 29 2018 20:33
    SuperOP535 synchronize #94
  • Nov 29 2018 20:31
    SuperOP535 synchronize #94
  • Nov 29 2018 20:29
    SuperOP535 opened #94
  • Nov 29 2018 18:31
    SuperOP535 closed #93
  • Nov 29 2018 08:56
    SuperOP535 opened #93
  • Nov 25 2018 15:53

    rom1504 on master

    update chat badges (compare)

  • Nov 15 2018 13:29
    SuperOP535 opened #92
  • Nov 15 2018 13:23
    SuperOP535 commented #75
  • Nov 15 2018 11:18
    rom1504 commented #75
  • Nov 15 2018 10:18
    SuperOP535 commented #75
  • Nov 13 2018 01:17
    rom1504 commented #75
  • Nov 12 2018 23:19
    rom1504 closed #85
  • Nov 12 2018 23:19
    rom1504 commented #85
  • Nov 12 2018 23:19

    rom1504 on master

    Added LICENSE file (compare)

Hans Elias J.
@hansihe
when writing it would take "packet_read" and map it to 0
switch is not capable of doing that
Robin Lambertz
@roblabla
It could
NVM
it can't.
My brain is not working
Hans Elias J.
@hansihe
roblabla: when a container only has a single non-virtual field in it, you can mark the container as virtual
that removes the container from the output, the single non-virtual field replaces it in the serialized/deserialized data
Robin Lambertz
@roblabla
ah, so it's like the old anon ?
William Gaylord
@wgaylord
but would the non-virtual field not then be virtual? Unless there is a non-virtual tag or something on the field....
Robin Lambertz
@roblabla
kinda sorta
Hans Elias J.
@hansihe
no, not really
well, the other way around kinda
Robin Lambertz
@roblabla
hmm, yeah I get it
Hans Elias J.
@hansihe
it's a way of defining something like a length prefixed array type without specific built in language features
William Gaylord
@wgaylord
how does it know that the field is not virtual along with the container?
Hans Elias J.
@hansihe
it just helps with composability really
Robin Lambertz
@roblabla
@chibill virtual just means you don't provide it data directly
instead it is derived from another field.
Hans Elias J.
@hansihe
chibill: notice that there are fields and there are virtual_fields? fields are not virtual, virtual_fields are virtual
Robin Lambertz
@roblabla
E.G. when writing, you don't have to specify the length of an array yourself. That's because "count" is a virtual field.
William Gaylord
@wgaylord
so why not have virtual_container?
Robin Lambertz
@roblabla
valid question.
Hans Elias J.
@hansihe
because the container has the exact same semantics when specifying types wither it is virtual or not
William Gaylord
@wgaylord
Also I did not notice virtual_field because its not in the example.
Hans Elias J.
@hansihe
but yeah, that's a design decision
Robin Lambertz
@roblabla
@chibill it is
in the pds
Hans Elias J.
@hansihe
yep
look for it
it's there
Robin Lambertz
@roblabla
virtual_field is just "virtual" in the JSON
which is interesting...
William Gaylord
@wgaylord
so why not just add virtual into the type args... As thats what it will be in the json...
why make a new type?
Robin Lambertz
@roblabla
It's not really a new type
it replaces count
and makes it more generic
William Gaylord
@wgaylord
virtual_field is a new type if you look how the pds is. Each type is defined by a syntax like <type>()
But its a type that only exists in the pds
Robin Lambertz
@roblabla
virtual_field is not a type in the pds, but it's a type in the JSON actually
in the pds, there are two kinds container fields. field and virtual_field.
In the JSON, field is just a pass-through, while virtual_field transforms into a type
Hans Elias J.
@hansihe
Question, if you think protodef is so stupid in every single way, why don't you just design your own version/use something else? If you come up with something better I would definitely use it.
William Gaylord
@wgaylord
I have never said it was stupid.
I am just questioning the questionable stuff that does not make sense.
Hans Elias J.
@hansihe
It would help a lot if you could come up with alternative solutions instead of JUST criticizing. We are aware there is a lot of complexity in some places, but it's that way because we believe it to be necessary
Robin Lambertz
@roblabla
Of all the points you gave, only one was valid : the new version make the semantics of container in the JSON dirtier
And when you did come up with that point, you didn't try to help make it better. You just criticized.