search

ShayBox / Mon2Cam

Workaround for multi-monitor Discord screensharing
MIT License
337 stars 33 forks source link

Add stderr output to exec #40

Closed botiapa closed 3 years ago

botiapa commented 3 years ago

Fixes https://github.com/ShayBox/Mon2Cam/issues/39

For some reason the console.log() command cuts off the output string. Example:

https://i.imgur.com/x2XNRYv.png

ShayBox commented 3 years ago

Yeah console.log restricts length of elements, you'll want to pass it into Deno.inspect first to extend everything to the max length, probably some options you'll have to set to increase object and array lengths too

botiapa commented 3 years ago

I couldn't find the max string length option. In node it's called maxStringLength, but I couldn't find the deno version of it. (sources: nodejs util documention deno options that I found ) I'd appreciate if you could you have a look at it yourself. If you can't find anything, or don't want to, we can always just use multiple prints.

ShayBox commented 3 years ago

It probably already expands strings, but if not that's easily done with a custom type formatter

On Fri, Aug 28, 2020, 9:59 AM killswitch notifications@github.com wrote:

I couldn't find the max string length option. In node it's called maxStringLength, but I couldn't find the deno version of it. (sources: nodejs util documention https://nodejs.org/api/util.html#util_util_inspect_object_options deno options that I found https://i.imgur.com/cptTDNg.png ) I'd appreciate if you could you have a look at it yourself. If you can't find anything, or don't want to, we can always just use multiple prints.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ShayBox/Mon2Cam/pull/40#issuecomment-682590830, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACIQTLB4GHHP26FXQIFGVILSC6Z3RANCNFSM4QL7CCWQ .