Skip to content

Commit

Permalink
Merge pull request #384 from xushiwei/q
Browse files Browse the repository at this point in the history
patch internal/reflectlite; demo: sort.Ints
  • Loading branch information
xushiwei authored Jun 21, 2024
2 parents b6e5980 + b93fc3f commit 67014ae
Show file tree
Hide file tree
Showing 7 changed files with 70 additions and 14 deletions.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -208,6 +208,7 @@ Here are the Go packages that can be imported correctly:
* [math](https://pkg.go.dev/math)
* [math/bits](https://pkg.go.dev/math/bits)
* [math/cmplx](https://pkg.go.dev/math/cmplx)
* [sort](https://pkg.go.dev/sort)
* [strconv](https://pkg.go.dev/strconv)
* [sync/atomic](https://pkg.go.dev/sync/atomic)
* [sync](https://pkg.go.dev/sync) (partially)
Expand Down
11 changes: 11 additions & 0 deletions _demo/sortdemo/sort.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
package main

import "sort"

func main() {
vals := []int{32, 58, 25, 92, 45, 78}
sort.Ints(vals)
for _, v := range vals {
println(v)
}
}
5 changes: 3 additions & 2 deletions cl/import.go
Original file line number Diff line number Diff line change
Expand Up @@ -518,14 +518,15 @@ func ignoreName(name string) bool {
return true
}
*/
return (strings.HasPrefix(name, "internal/") && !supportedInternal(name)) ||
const internal = "internal/"
return (strings.HasPrefix(name, internal) && !supportedInternal(name[len(internal):])) ||
strings.HasPrefix(name, "crypto/") || strings.HasPrefix(name, "runtime/") ||
strings.HasPrefix(name, "arena.") || strings.HasPrefix(name, "maps.") ||
strings.HasPrefix(name, "time.") || strings.HasPrefix(name, "plugin.")
}

func supportedInternal(name string) bool {
return strings.HasPrefix(name, "internal/bytealg.")
return strings.HasPrefix(name, "bytealg.") || strings.HasPrefix(name, "reflectlite.")
}

// -----------------------------------------------------------------------------
Expand Down
25 changes: 13 additions & 12 deletions internal/build/build.go
Original file line number Diff line number Diff line change
Expand Up @@ -750,18 +750,19 @@ func canSkipToBuild(pkgPath string) bool {
type none struct{}

var hasAltPkg = map[string]none{
"errors": {},
"internal/bytealg": {},
"io": {},
"io/fs": {},
"math": {},
"math/cmplx": {},
"reflect": {},
"sync": {},
"sync/atomic": {},
"syscall": {},
"os": {},
"runtime": {},
"errors": {},
"internal/bytealg": {},
"internal/reflectlite": {},
"io": {},
"io/fs": {},
"math": {},
"math/cmplx": {},
"reflect": {},
"sync": {},
"sync/atomic": {},
"syscall": {},
"os": {},
"runtime": {},
}

func check(err error) {
Expand Down
22 changes: 22 additions & 0 deletions internal/lib/internal/reflectlite/reflectlite.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
/*
* Copyright (c) 2024 The GoPlus Authors (goplus.org). All rights reserved.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

package reflectlite

// llgo:skipall
import (
_ "unsafe"
)
16 changes: 16 additions & 0 deletions internal/lib/reflect/makefunc.go
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
/*
* Copyright (c) 2024 The GoPlus Authors (goplus.org). All rights reserved.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

// Copyright 2012 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.
Expand Down
4 changes: 4 additions & 0 deletions internal/lib/reflect/value.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,10 @@
* limitations under the License.
*/

// Copyright 2009 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.

package reflect

import (
Expand Down

0 comments on commit 67014ae

Please sign in to comment.