Repository navigation

Avoid calls to (*os.File).Fd() when calling ioctl - #214

Closed

WeidiDeng wants to merge 13 commits into

creack:masterfrom

WeidiDeng:master

Closed

Avoid calls to (*os.File).Fd() when calling ioctl#214
WeidiDeng wants to merge 13 commits into
creack:masterfrom
WeidiDeng:master

Commits on Mar 11, 2026
- Initial plan
  Copilot
  committed
  01c1397
- refactor: change ioctl signature to use any for ptr parameter
  
  Copilot
  and
  WeidiDeng
  committed
  9b21f6e
- fix: defer uintptr conversion to syscall.Syscall call site via unsafe.Pointer
  
  Copilot
  and
  WeidiDeng
  committed
  b396826
- fix: remove any param; use unsafe.Pointer at each ioctl call site
  
  Copilot
  and
  WeidiDeng
  committed
  365da9a
- refactor: ioctl ptr parameter from uintptr to any; convert any→uintptr at syscall.Syscall site
  
  Copilot
  and
  WeidiDeng
  committed
  38adeb0
- refactor: inline any→uintptr via reflect.ValueOf(ptr).UnsafePointer() at syscall call site; remove ptrToUintptr helper
  
  Copilot
  and
  WeidiDeng
  committed
  adf8f70
- refactor: use if/else to pass uintptr(0) or uintptr(reflect.ValueOf(ptr).UnsafePointer()) inline at syscall call site
  
  Copilot
  and
  WeidiDeng
  committed
  548280c
- remove empty file created by copilot
  WeidiDeng
  authored
  afb4333
- use non blocking ioctl
  WeidiDeng
  authored
  4c910f0
- fix legacy support
  WeidiDeng
  authored
  052587b
- exclude ioctl_inner from legacy go
  WeidiDeng
  authored
  0a3f138
- remove syscall.SetNonblock when testing and in README
  WeidiDeng
  authored
  6d89a20
- Merge pull request #2 from WeidiDeng/copilot/refactor-ioctl-usage
  WeidiDeng
  authored
  820383e