Merge pull request #113 from exoego/deprecated-in-0.9.0

Remove items deprecated in v0.9.0

Author: exoego

app/current/src/main/scala/io/scalajs/nodejs/Global.scala

@@ -86,7 +86,7 @@ class Buffer private[this] () extends Uint8Array( /* dummy to trick constructor
     *
     * @see [[https://nodejs.org/api/buffer.html#buffer_buf_entries]]
     */
-  def entries(): io.scalajs.collection.Iterator[js.Array[Int]] = js.native
+  def entries(): js.Iterator[js.Array[Int]] = js.native
 
   /**
     * Returns true if both buf and otherBuffer have exactly the same bytes, false otherwise.
@@ -139,7 +139,7 @@ class Buffer private[this] () extends Uint8Array( /* dummy to trick constructor
     * @return an [[Iterator]]
     * @example buf.keys()
     */
-  def keys(): io.scalajs.collection.Iterator[Int] = js.native
+  def keys(): js.Iterator[Int] = js.native
 
   /**
     * The largest size allowed for a single Buffer instance.
@@ -486,7 +486,7 @@ class Buffer private[this] () extends Uint8Array( /* dummy to trick constructor
     * @return an iterator for buf values (bytes)
     * @example buf.values()
     */
-  def values(): io.scalajs.collection.Iterator[Int] = js.native
+  def values(): js.Iterator[Int] = js.native
 
   /**
     * Writes string to buf at offset according to the character encoding in encoding. The length parameter is

app/current/src/main/scala/io/scalajs/nodejs/buffer/Buffer.scala

@@ -226,7 +226,7 @@ class Console protected () extends js.Object {
 }
 
 /**
-  * A global `Console` instance configured to write to [[io.scalajs.nodejs.process.stdout]] and [[io.scalajs.nodejs.process.stderr]].
+  * A global `Console` instance configured to write to [[io.scalajs.nodejs.process.Process.stdout]] and [[io.scalajs.nodejs.process.Process.stderr]].
   */
 @js.native
 @JSGlobal("console")

app/current/src/main/scala/io/scalajs/nodejs/console_module/Console.scala

@@ -38,10 +38,6 @@ package object fs {
     * @param instance the given [[Fs file system]] instance
     */
   implicit final class FsExtensions(private val instance: Fs) extends AnyVal {
-    @deprecated("Use Fs directly instead of Fs.fs", "0.9.0")
-    @inline
-    def fs: Fs = instance
-
     @inline
     def accessFuture(path: Buffer | String): Future[Unit] = {
       promiseWithError0[FileIOError](instance.access(path, _))

app/current/src/main/scala/io/scalajs/nodejs/fs/package.scala

@@ -3,7 +3,6 @@ package io.scalajs
 import com.thoughtworks.enableIf
 import io.scalajs.nodejs.timers._
 
-import scala.concurrent.duration.FiniteDuration
 import scala.scalajs.js
 import scala.scalajs.js.annotation.JSGlobal
 import scala.scalajs.js.|
@@ -17,12 +16,6 @@ package object nodejs {
   //      Type Definitions
   /////////////////////////////////////////////////////////////////////////////////
 
-  @deprecated("use process.Process instead", "0.9.0")
-  type Process = process.Process
-
-  @deprecated("use process.Environment instead", "0.9.0")
-  type Environment = process.Environment
-
   type EventType = String
 
   /**
@@ -56,14 +49,6 @@ package object nodejs {
   /////////////////////////////////////////////////////////////////////////////////
   //      Built-in Properties
   /////////////////////////////////////////////////////////////////////////////////
-  @deprecated("Use console_module.Console. Every module stays each own package.", "0.9.0")
-  type Console = console_module.Console
-
-  @deprecated("Use console_module.Console. Every module stays each own package.", "0.9.0")
-  type ConsoleDirOptions = console_module.ConsoleDirOptions
-
-  @deprecated("Use console_module.Console. Every module stays each own package.", "0.9.0")
-  val console = console_module.Console
 
   /**
     * The directory name of the current module. This the same as the path.dirname() of the [[__filename]].
@@ -77,11 +62,6 @@ package object nodejs {
     */
   def __filename: String = js.Dynamic.global.__filename.asInstanceOf[String]
 
-  @js.native
-  @JSGlobal("global")
-  @deprecated("Use objects in io.scalajs.nodejs", "0.9.0")
-  object global extends Global
-
   @js.native
   @enableIf(io.scalajs.nodejs.internal.CompilerSwitches.gteNodeJs12)
   @JSGlobal("queueMicrotask")
@@ -116,33 +96,4 @@ package object nodejs {
   @js.native
   @JSGlobal("setTimeout")
   object setTimeout extends SetTimeout
-
-  /////////////////////////////////////////////////////////////////////////////////
-  //      Implicit Conversions
-  /////////////////////////////////////////////////////////////////////////////////
-
-  /**
-    * Implicit conversion to translate durations into an integer
-    * @param duration the given [[FiniteDuration duration]]
-    * @return the time in milliseconds as an integer
-    */
-  @deprecated("Use io.scalajs.util.DurationHelper", "0.9.0")
-  implicit def duration2Int(duration: FiniteDuration): Int = duration.toMillis.toInt
-
-  /**
-    * Implicit conversion to translate durations into a double
-    * @param duration the given [[FiniteDuration duration]]
-    * @return the time in milliseconds as a double
-    */
-  @deprecated("Use io.scalajs.util.DurationHelper", "0.9.0")
-  implicit def duration2Double(duration: FiniteDuration): Double = duration.toMillis.toDouble
-
-  /**
-    * Implicit conversion to transform Node [[Error]]s to [[Exception]]s
-    * @param error the given [[Error]]
-    * @return the resulting [[Exception]]
-    */
-  @deprecated("Use toException extension method from io.scalajs.util.NodeJSConverters._", "0.9.0")
-  implicit def error2Exception(error: Error): Exception =
-    io.scalajs.util.NodeJSConverters.ErrorExtension(error).toException()
 }

app/current/src/main/scala/io/scalajs/nodejs/package.scala

@@ -33,13 +33,3 @@ trait Environment extends js.Object {
   @JSBracketAccess
   def update(key: String, value: String): Unit = js.native
 }
-
-object Environment {
-  implicit final class EnvironmentExtension(private val env: Environment) extends AnyVal {
-    @deprecated("Use env(key)", "0.9.0")
-    @inline def get(key: String): Option[String] = env(key).toOption
-
-    @deprecated("Use env(key)", "0.9.0")
-    @inline def contains(key: String): Boolean = env(key).isDefined
-  }
-}

app/current/src/main/scala/io/scalajs/nodejs/process/Environment.scala

@@ -1,7 +1,6 @@
 package io.scalajs.nodejs
 
 import com.thoughtworks.enableIf
-import io.scalajs.nodejs.tty.{ReadStream, WriteStream}
 import io.scalajs.util.PromiseHelper.promiseWithError1
 
 import scala.concurrent.Future
@@ -14,120 +13,6 @@ package object process {
   type ExitCode         = Int
   type SendHandle       = net.Socket | net.Server
 
-  @deprecated("use Process object instead", "0.9.0")
-  def allowedNodeEnvironmentFlags: EnvironmentFlags = Process.allowedNodeEnvironmentFlags
-  @deprecated("use Process object instead", "0.9.0")
-  def arch: String = Process.arch
-  @deprecated("use Process object instead", "0.9.0")
-  def argv: js.Array[String] = Process.argv
-  @deprecated("use Process object instead", "0.9.0")
-  def argv0: String = Process.argv0
-  @deprecated("use Process object instead", "0.9.0")
-  def channel: js.UndefOr[js.Object] = Process.channel
-  @deprecated("use Process object instead", "0.9.0")
-  def config: ProcessConfig = Process.config
-  @deprecated("use Process object instead", "0.9.0")
-  def connected: js.UndefOr[Boolean] = Process.connected
-  @deprecated("use Process object instead", "0.9.0")
-  def debugPort: Integer = Process.debugPort
-  @deprecated("use Process object instead", "0.9.0")
-  def env: Environment = Process.env
-  @deprecated("use Process object instead", "0.9.0")
-  def execArgv: js.Array[String] = Process.execArgv
-  @deprecated("use Process object instead", "0.9.0")
-  def execPath: String = Process.execPath
-  @deprecated("use Process object instead", "0.9.0")
-  def exitCode: Int = Process.exitCode
-  @deprecated("use Process object instead", "0.9.0")
-  def features: js.Dictionary[Boolean] = Process.features
-  @deprecated("use Process object instead", "0.9.0")
-  def mainModule: js.UndefOr[js.Any] = Process.mainModule
-  @deprecated("use Process object instead", "0.9.0")
-  def moduleLoadList: js.Array[String] = Process.moduleLoadList
-  @deprecated("use Process object instead", "0.9.0")
-  def pid: Int = Process.pid
-  @deprecated("use Process object instead", "0.9.0")
-  def platform: String = Process.platform
-  @deprecated("use Process object instead", "0.9.0")
-  def release: ReleaseInfo = Process.release
-  @deprecated("use Process object instead", "0.9.0")
-  def stderr: WriteStream = Process.stderr
-  @deprecated("use Process object instead", "0.9.0")
-  def stdin: ReadStream = Process.stdin
-  @deprecated("use Process object instead", "0.9.0")
-  def stdout: WriteStream = Process.stdout
-  @deprecated("use Process object instead", "0.9.0")
-  def title: String = Process.title
-  @deprecated("use Process object instead", "0.9.0")
-  def version: String = Process.version
-  @deprecated("use Process object instead", "0.9.0")
-  def versions: ComponentVersion = Process.versions
-
-  @deprecated("use Process object instead", "0.9.0")
-  def abort(): Unit = Process.abort()
-  @deprecated("use Process object instead", "0.9.0")
-  def chdir(directory: String): Unit = Process.chdir(directory)
-  @deprecated("use Process object instead", "0.9.0")
-  def cwd(): String = Process.cwd()
-  @deprecated("use Process object instead", "0.9.0")
-  def disconnect(): js.Any = Process.disconnect()
-  @deprecated("use Process object instead", "0.9.0")
-  def emitWarning(warning: String, name: String = null, code: String = null, ctor: js.Function = null): Unit =
-    Process.emitWarning(warning, name, code, ctor)
-  @deprecated("use Process object instead", "0.9.0")
-  def exit(code: Int = 0): Unit = Process.exit(code)
-  @deprecated("use Process object instead", "0.9.0")
-  def getegid(): Int = Process.getegid()
-  @deprecated("use Process object instead", "0.9.0")
-  def geteuid(): Int = Process.geteuid()
-  @deprecated("use Process object instead", "0.9.0")
-  def getgid(): Int = Process.getgid()
-  @deprecated("use Process object instead", "0.9.0")
-  def getgroups(): js.Array[Int] = Process.getgroups()
-  @deprecated("use Process object instead", "0.9.0")
-  def getuid(): Int = Process.getuid()
-  @deprecated("use Process object instead", "0.9.0")
-  def hrtime(time: js.Array[Int] = null): js.Array[Int] = Process.hrtime(time)
-  @deprecated("use Process object instead", "0.9.0")
-  def initgroups(user: String, extra_group: String): js.Array[Int] = Process.initgroups(user, extra_group)
-  @deprecated("use Process object instead", "0.9.0")
-  def kill(pid: Int, signal: String = null): Unit = Process.kill(pid, signal)
-  @deprecated("use Process object instead", "0.9.0")
-  def memoryUsage(): MemoryUsage = Process.memoryUsage()
-  @deprecated("use Process object instead", "0.9.0")
-  def nextTick(callback: js.Function0[Any], args: js.Any*): Unit = Process.nextTick(callback, args)
-  @deprecated("use Process object instead", "0.9.0")
-  def openStdin(): ReadStream = Process.openStdin()
-  @deprecated("use Process object instead", "0.9.0")
-  def send(message: js.Any, sendHandle: SendHandle, options: TransferOptions, callback: js.Function): Boolean =
-    Process.send(message, sendHandle, options, callback)
-  @deprecated("use Process object instead", "0.9.0")
-  def send(message: js.Any, sendHandle: SendHandle, options: TransferOptions): Boolean =
-    Process.send(message, sendHandle, options)
-  @deprecated("use Process object instead", "0.9.0")
-  def send(message: js.Any, sendHandle: SendHandle, callback: js.Function): Boolean =
-    Process.send(message, sendHandle, callback)
-  @deprecated("use Process object instead", "0.9.0")
-  def send(message: js.Any, callback: js.Function): Boolean = Process.send(message, callback)
-  @deprecated("use Process object instead", "0.9.0")
-  def send(message: js.Any): Boolean = Process.send(message)
-  @deprecated("use Process object instead", "0.9.0")
-  def setegid(id: Int): Unit = Process.setegid(id)
-  @deprecated("use Process object instead", "0.9.0")
-  def seteuid(id: Int): Unit = Process.seteuid(id)
-  @deprecated("use Process object instead", "0.9.0")
-  def setgid(id: Int): Unit = Process.setgid(id)
-  @deprecated("use Process object instead", "0.9.0")
-  def setgroups(groups: js.Array[Int] | js.Array[String] | js.Array[String | Int]): Unit = Process.setgroups(groups)
-  @deprecated("use Process object instead", "0.9.0")
-  def setuid(id: Int): Unit = Process.setuid(id)
-  @deprecated("use Process object instead", "0.9.0")
-  def umask(mask: Int): Int = Process.umask(mask)
-  @deprecated("use Process object instead", "0.9.0")
-  def umask(): Int = Process.umask()
-  @deprecated("use Process object instead", "0.9.0")
-  def uptime(): Int = Process.uptime()
-
   /**
     * Process Object Extensions
     * @param process the given [[Process process]]

app/current/src/main/scala/io/scalajs/nodejs/process/package.scala

@@ -152,19 +152,6 @@ sealed trait IReadable extends LegacyStream {
     */
   def push(chunk: Uint8Array): Boolean = js.native
 
-  /**
-    * The read() method pulls some data out of the internal buffer and returns it. If there is no data available,
-    * then it will return null.
-    * <p/>If you pass in a size argument, then it will return that many bytes. If size bytes are not available,
-    * then it will return null, unless we've ended, in which case it will return the data remaining in the buffer.
-    * <p/>If you do not specify a size argument, then it will return all the data in the internal buffer.
-    * <p/>This method should only be called in paused mode. In flowing mode, this method is called automatically
-    * until the internal buffer is drained.
-    * @example readable.read([size])
-    */
-  @deprecated("Use readAsXXXX instead", "0.9.0")
-  def read[T](size: Int = js.native): T = js.native
-
   @JSName("read")
   def readAsString(size: Int = js.native): String = js.native
   @JSName("read")
@@ -335,24 +322,6 @@ object IReadable {
       }
     }
 
-    @inline
-    @deprecated("Use iteratorAsXXXX instead", "0.9")
-    def iterator[T]: scala.Iterator[T] = new scala.Iterator[T] {
-      var result: T = readable.read[T]()
-
-      override def hasNext: Boolean = result != null
-
-      override def next(): T = {
-        val value = result
-        result = readable.read[T]()
-        value
-      }
-    }
-
-    @inline
-    @deprecated("Use readAsXXXXOption instead", "0.9")
-    def readOption[A](): Option[A] = Option(readable.read[A]())
-
     @inline
     def readAsObjectOption(): Option[js.Any] = Option(readable.readAsObject())
 

app/current/src/main/scala/io/scalajs/nodejs/stream/Stream.scala

@@ -67,7 +67,7 @@ class URLSearchParams() extends js.Object {
     * a JavaScript Array. The first item of the Array is the name, the second item of the Array is the value.
     * @return an iterable of an array of results
     */
-  def entries(): io.scalajs.collection.Iterator[js.Tuple2[String, String]] = js.native
+  def entries(): js.Iterator[js.Tuple2[String, String]] = js.native
 
   /**
     * Iterates over each name-value pair in the query and invokes the given function.
@@ -96,7 +96,7 @@ class URLSearchParams() extends js.Object {
     */
   def has(name: String): Boolean = js.native
 
-  def keys(): io.scalajs.collection.Iterator[String] = js.native
+  def keys(): js.Iterator[String] = js.native
 
   /**
     * Sets the value in the URLSearchParams object associated with name to value. If there are any pre-existing
@@ -116,6 +116,6 @@ class URLSearchParams() extends js.Object {
   /**
     * Returns an ES6 Iterator over the values of each name-value pair.
     */
-  def values(): io.scalajs.collection.Iterator[String] = js.native
+  def values(): js.Iterator[String] = js.native
 
 }