Stream: git-wasmtime

Topic: wasmtime / issue #6202 Clarify the representation of `icm...


view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2023 at 19:23):

jameysharp commented on issue #6202:

I think this is a very helpful improvement. I believe the same description would also apply to fcmp, if you want to add it there too.

This is an extremely nit-picky detail but I'd prefer "i.e." written with periods. It doesn't actually matter but it bothers me anyway. :sweat_smile:

I'd like @cfallin's confirmation that this description is correct but I believe it is.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2023 at 19:38):

kevaundray commented on issue #6202:

I think this is a very helpful improvement. I believe the same description would also apply to fcmp, if you want to add it there too.

This is an extremely nit-picky detail but I'd prefer "i.e." written with periods. It doesn't actually matter but it bothers me anyway. :sweat_smile:

I'd like @cfallin's confirmation that this description is correct but I believe it is.

Now that you've brought it to my attention, I can no longer ignore the missing periods :sweat_smile:

view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2023 at 19:52):

cfallin commented on issue #6202:

I'd like @cfallin's confirmation that this description is correct but I believe it is.

LGTM!


Last updated: Nov 22 2024 at 17:03 UTC