Re: [OpenSmalltalk/opensmalltalk-vm] Mac CameraPlugin: Add safety, only copying the minimum of the size of the c (02a2747)

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: [OpenSmalltalk/opensmalltalk-vm] Mac CameraPlugin: Add safety, only copying the minimum of the size of the c (02a2747)

David T Lewis
 

Was that intentional?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

<script type="application/ld+json">[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/02a274782dfa1747a846950423ec2bb555c282e9#commitcomment-40707585", "url": "https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/02a274782dfa1747a846950423ec2bb555c282e9#commitcomment-40707585", "name": "View Commit" }, "description": "View this Commit on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]</script>
Reply | Threaded
Open this post in threaded view
|

Re: [OpenSmalltalk/opensmalltalk-vm] Mac CameraPlugin: Add safety, only copying the minimum of the size of the c (02a2747)

David T Lewis
 

Can we please not have one-armed two-line ifs? :( I know you like that style but its still dangerous…


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

<script type="application/ld+json">[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/02a274782dfa1747a846950423ec2bb555c282e9#commitcomment-40707594", "url": "https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/02a274782dfa1747a846950423ec2bb555c282e9#commitcomment-40707594", "name": "View Commit" }, "description": "View this Commit on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]</script>