Giter Site home page Giter Site logo

macex raises on () about janet HOT 8 CLOSED

ianthehenry avatar ianthehenry commented on May 25, 2024
macex raises on ()

from janet.

Comments (8)

sogaiu avatar sogaiu commented on May 25, 2024 2

Looks like in the most recent janet (11d7af3), the dotup bit is this line.

from janet.

iacore avatar iacore commented on May 25, 2024 1

(discussed before) this very footgun-ish maybe we make () invalid instead of empty list

from janet.

sogaiu avatar sogaiu commented on May 25, 2024 1

Are you confident that that will have minimal to no effect on existing code?

from janet.

sogaiu avatar sogaiu commented on May 25, 2024 1

the inconsistency that () is not a "function call"

I think it depends on how one defines things.

There is at least one other Lisp-like that has a similar arrangement:

An empty list () evaluates to an empty list.

Non-empty Lists are considered calls to either special forms, macros, or functions. A call has the form (operator operands*).

via: https://clojure.org/reference/evaluation

By this definition, () isn't a call and at least when looking directly at such a definition, there doesn't appear to be an outright inconsistency to me.

from janet.

sogaiu avatar sogaiu commented on May 25, 2024

I don't suppose an appropriate fix is as simple as:

diff --git a/src/boot/boot.janet b/src/boot/boot.janet
index 9c5a2bd9..354f0f14 100644
--- a/src/boot/boot.janet
+++ b/src/boot/boot.janet
@@ -2141,7 +2141,9 @@
     (case (type x)
       :tuple (if (= (tuple/type x) :brackets)
                (tuple/brackets ;(map recur x))
-               (dotup x))
+               (if (next x)
+                 (dotup x)
+                 ()))
       :array (map recur x)
       :struct (table/to-struct (dotable x recur))
       :table (dotable x recur)
$ ./build/janet
Janet 1.32.1-11d7af3f linux/x64/gcc - '(doc)' for help
repl:1:> (macex '(if true () ()))
(if true () ())

from janet.

pepe avatar pepe commented on May 25, 2024

Something like that, I wanted to propose. :-).

from janet.

iacore avatar iacore commented on May 25, 2024

Are you confident that that will have minimal to no effect on existing code?

very not confident

the thing is, the inconsistency that () is not a "function call" will probably end up being discovered again.

there's a way to add warnings to bare () (suggest to switch to [])

from janet.

iacore avatar iacore commented on May 25, 2024

It's inconsistent because without any prior knowledge to lisp-like languages, one would deduct () (a) (a b) (a b c) to behave similarly. That's also the simplest way to write code to handle lispy expressions. In this case (macex) the inconsistency caused the mistake to happen.

People by default don't know, so I argue to make a language self-consistent rather than follow convention (requires knowledge).

from janet.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.