At least mention the intention of a block of assembly

cag · cag · commit 0bf6a8449237 · 2020-12-22T21:18:46.000-06:00
diff --git a/contracts/CPKFactoryFacade.sol b/contracts/CPKFactoryFacade.sol
@@ -38,6 +38,7 @@ contract CPKFactoryFacade {
         returns (bool)
     {
         address owner;
+        // the following assembly block extracts the owner from the signature data
         // solium-disable-next-line security/no-inline-assembly
         assembly {
             owner := and(

Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,7 @@ contract CPKFactoryFacade {`
`38`	`38`	`returns (bool)`
`39`	`39`	`{`
`40`	`40`	`address owner;`
	`41`	`+ // the following assembly block extracts the owner from the signature data`
`41`	`42`	`// solium-disable-next-line security/no-inline-assembly`
`42`	`43`	`assembly {`
`43`	`44`	`owner := and(`