Skip to content

Fix for ansi-test SXHASH.8#1

Merged
easye merged 2 commits into
armedbear:masterfrom
logicmoo:master
Oct 7, 2016
Merged

Fix for ansi-test SXHASH.8#1
easye merged 2 commits into
armedbear:masterfrom
logicmoo:master

Conversation

@TeamSPoon
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Collaborator

@easye easye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine for one commit, but I would prefer "best practice" conventions for Java code that separates the IF condition with whitespace and uses an explicit block with curly braces.

i.e. instead of

if(capacity == 0) return 0;

if (capacity == 0) {
return 0;
}

@easye easye merged commit 65b28c3 into armedbear:master Oct 7, 2016
easye pushed a commit that referenced this pull request Oct 13, 2016
launchctl setenv PATH /opt/local/bin:/opt/local/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin
illin:~$ ssh mevenson@abcl.org
Linux lisp.not.org 3.16.0-0.bpo.4-amd64 #1 SMP Debian 3.16.36-1+deb8u1~bpo70+1 (2016-09-08) x86_64

The programs included with the Debian GNU/Linux system are free software;
the exact distribution terms for each program are described in the
individual files in /usr/share/doc/*/copyright.

Debian GNU/Linux comes with ABSOLUTELY NO WARRANTY, to the extent
permitted by applicable law.
Last login: Mon Oct 10 13:57:49 2016 from 84-115-198-249.cable.dynamic.surfer.at
mevenson@lisp:~$ screen -DR









Fix CL:OPEN for :DIRECTION :INPUT (pipping)
Elias Pipping via #abcl <http://dpaste.com/2KYD7RV.txt>

If you take a look at open.lisp, you'll see that for :direction
:probe, if-does-not-exist :error/:create is handledyet for
:direction :input, only :error is, and for any other key, nothing
happens (the code uses `case` rather than `ecase`)
alanruttenberg added a commit to alanruttenberg/abcl that referenced this pull request Feb 14, 2017
….subclass(Object.class,t).method(ElementMatchers.named("toString"))"
alanruttenberg added a commit to alanruttenberg/abcl that referenced this pull request Feb 14, 2017
….subclass(Object.class,t).method(ElementMatchers.named("toString"))"
easye pushed a commit that referenced this pull request Jun 9, 2017
JAVAPARSER now "stacks" its macro definition on the existing one it
inherits from JSS.

Refactor all tests to just pollute CL-USER where necessary (strictly
speaking it shouldn't be necessary as all PROVE tests should be able
to use lexical bindings.

Document existence of Java field access via SHARPSIGN-QUOTATION_MARK,
e.g.

    (#"{java.lang.System}.{fileSeparator}")

and the new Java DSL

    (asdf:make :javaparser)
    (#"'#1"new ByteBuddy()
      .subclass(Object.class,t)
      .method(ElementMatchers.named("toString"))
      .intercept(FixedValue.value("Hello World!"))
      .make()
      .load(getClass().getClassLoader())
      .getLoaded()"
Ferada pushed a commit to Ferada/abcl that referenced this pull request May 2, 2019
alanruttenberg added a commit to alanruttenberg/abcl that referenced this pull request Dec 17, 2019
…the armedbear#1".." reader, which allows java expressions e.g. armedbear#1"System.out.printLn("foo")"
alanruttenberg added a commit to alanruttenberg/abcl that referenced this pull request Nov 5, 2021
Other minor fixes in invoke.lisp
Engage the armedbear#1".." reader, which allows java expressions e.g. armedbear#1"System.out.printLn("foo")"
@alanruttenberg alanruttenberg mentioned this pull request Dec 1, 2021
alanruttenberg added a commit to alanruttenberg/abcl that referenced this pull request Feb 22, 2022
alanruttenberg added a commit to alanruttenberg/abcl that referenced this pull request Apr 6, 2022
…able. Have invoke load javaparser first time armedbear#1 is used.
easye pushed a commit that referenced this pull request Apr 6, 2022
…able. Have invoke load javaparser first time #1 is used.
@alanruttenberg alanruttenberg mentioned this pull request Apr 11, 2022
alejandrozf pushed a commit to alejandrozf/abcl that referenced this pull request Sep 2, 2023
@alejandrozf alejandrozf mentioned this pull request Sep 2, 2023
easye pushed a commit that referenced this pull request Sep 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants