public inbox for kawa@sourceware.org
 help / color / mirror / Atom feed
* Re: <unknown>
       [not found] <CAMFYt2bEiKK4+hc6wYY150Jqgr44Zca=EieRXuEoZw1w4N4vyg@mail.gmail.com>
@ 2022-09-13 14:56 ` Per Bothner
       [not found]   ` <CAMFYt2a0_dbOmU9MesdMfoDDcH42b2MBJ9wBN+dHc537DSLGUg@mail.gmail.com>
  0 siblings, 1 reply; 2+ messages in thread
From: Per Bothner @ 2022-09-13 14:56 UTC (permalink / raw)
  To: Panicz Maciej Godek, kawa

[-- Attachment #1: Type: text/plain, Size: 132 bytes --]

Try the attached patch to see if you get a more helpful error message.

-- 
	--Per Bothner
per@bothner.com   http://per.bothner.com/

[-- Attachment #2: cl-err-patch.txt --]
[-- Type: text/plain, Size: 1066 bytes --]

diff --git a/gnu/expr/ClassExp.java b/gnu/expr/ClassExp.java
index b8ce460b2..40b45f816 100644
--- a/gnu/expr/ClassExp.java
+++ b/gnu/expr/ClassExp.java
@@ -643,10 +643,11 @@ public class ClassExp extends LambdaExp
                             generateBridgeMethod(comp, impl, ptypes, rtype);
                         } else {
                             // FIXME - need better error message!
-                            String msg = vec.size() == 0
-                                ? "missing implementation for "
-                                : "ambiguous implementation for ";
-                            comp.error('e', msg+meth);
+                            String msg = "non-abstract class "+getName();
+                            msg += vec.size() == 0
+                                ? " is missing implementation for "
+                                : " has ambiguous implementation for ";
+                            comp.error('e', msg+meth, this);
                         }
                     } else {
                         Method impl

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: <unknown>
       [not found]   ` <CAMFYt2a0_dbOmU9MesdMfoDDcH42b2MBJ9wBN+dHc537DSLGUg@mail.gmail.com>
@ 2022-09-19 10:45     ` Per Bothner
  0 siblings, 0 replies; 2+ messages in thread
From: Per Bothner @ 2022-09-19 10:45 UTC (permalink / raw)
  To: Panicz Maciej Godek; +Cc: kawa

On 9/16/22 12:24, Panicz Maciej Godek wrote:
> Hi,
> sorry for the delayed response.
> I managed to figure out the problem by other means.
> However, I just checked to see whether your patch would have helped -- and it worked!
> 
> The error message that I got was the following:
> 
> <unknown>: non-abstract class HeadTailSeparator is missing implementation for Indexable.cursorUnder$St(gnu.math.RealNum,gnu.math.RealNum,java.lang.Object)java.lang.Object

I checked in an improvement of this this.  My test program now emits:

/tmp/x.scm:2:2: non-abstract class Foo is missing implementation for java.lang.Runnable.run()void
/tmp/x.scm:5:4: non-abstract object is missing implementation for java.lang.Runnable.run()void

-- 
	--Per Bothner
per@bothner.com   http://per.bothner.com/

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-09-19 10:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CAMFYt2bEiKK4+hc6wYY150Jqgr44Zca=EieRXuEoZw1w4N4vyg@mail.gmail.com>
2022-09-13 14:56 ` <unknown> Per Bothner
     [not found]   ` <CAMFYt2a0_dbOmU9MesdMfoDDcH42b2MBJ9wBN+dHc537DSLGUg@mail.gmail.com>
2022-09-19 10:45     ` <unknown> Per Bothner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).