grace: fix empty copyOut (#97)

BoYanZh · criyle · web-flow · commit b0c85a648166 · 2024-03-04T15:12:53.000+08:00
* fix: remove empty copyOut

* reserve capacity for copyOut

---------

Co-authored-by: Yang Gao &lt;6821729+criyle@users.noreply.github.com&gt;
diff --git a/cmd/go-judge-shell/grpc.go b/cmd/go-judge-shell/grpc.go
@@ -181,7 +181,7 @@ func convertPBCopyIn(copyIn map[string]model.CmdFile) map[string]*pb.Request_Fil
 }
 
 func convertPBCopyOut(copyOut []string) []*pb.Request_CmdCopyOutFile {
-	rt := make([]*pb.Request_CmdCopyOutFile, len(copyOut))
+	rt := make([]*pb.Request_CmdCopyOutFile, 0, len(copyOut))
 	for _, n := range copyOut {
 		optional := false
 		if strings.HasSuffix(n, "?") {

Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,7 @@ func convertPBCopyIn(copyIn map[string]model.CmdFile) map[string]*pb.Request_Fil`
`181`	`181`	`}`
`182`	`182`
`183`	`183`	`func convertPBCopyOut(copyOut []string) []*pb.Request_CmdCopyOutFile {`
`184`		`- rt := make([]*pb.Request_CmdCopyOutFile, len(copyOut))`
	`184`	`+ rt := make([]*pb.Request_CmdCopyOutFile, 0, len(copyOut))`
`185`	`185`	`for _, n := range copyOut {`
`186`	`186`	`optional := false`
`187`	`187`	`if strings.HasSuffix(n, "?") {`