From demmeln at in.tum.de Sun Feb 14 19:30:53 2010 From: demmeln at in.tum.de (Nikolaus Demmel) Date: Sun, 14 Feb 2010 20:30:53 +0100 Subject: [cl-store-devel] CREATE-FLOAT-VALUE defined twice. Message-ID: <8BB2F7E9-910F-44A3-A863-26CF6D9A3557@in.tum.de> Hi, the function CREATE-FLOAT-VALUE is defined once in "default-backend.lisp" and once in the implementation specific files. This results in a warning about redefining said function when loading the system. Is there a reason the default definition is there (it does nothing) and would it be possible to remove it and just leave the note saying that each implementation must provide it? I'd be happy to provide a patch. Btw thanks for this great library! I found it to be very usefull (especially the extensibility and the circularity handling). Regards, Niko -------------- next part -------------- A non-text attachment was scrubbed... Name: smime.p7s Type: application/pkcs7-signature Size: 4623 bytes Desc: not available URL: From rosssd at gmail.com Tue Feb 16 09:32:39 2010 From: rosssd at gmail.com (Sean Ross) Date: Tue, 16 Feb 2010 09:32:39 +0000 Subject: [cl-store-devel] CREATE-FLOAT-VALUE defined twice. In-Reply-To: <8BB2F7E9-910F-44A3-A863-26CF6D9A3557@in.tum.de> References: <8BB2F7E9-910F-44A3-A863-26CF6D9A3557@in.tum.de> Message-ID: Hi Nikolaus, A patch which removes the warning would be appreciated. Glad you are finding the library useful. Cheers, Sean. On 14 Feb 2010, at 19:30, Nikolaus Demmel wrote: > Hi, > > the function CREATE-FLOAT-VALUE is defined once in "default-backend.lisp" and once in the implementation specific files. This results in a warning about redefining said function when loading the system. Is there a reason the default definition is there (it does nothing) and would it be possible to remove it and just leave the note saying that each implementation must provide it? I'd be happy to provide a patch. > > Btw thanks for this great library! I found it to be very usefull (especially the extensibility and the circularity handling). > > Regards, > Niko_______________________________________________ > cl-store-devel mailing list > cl-store-devel at common-lisp.net > http://common-lisp.net/cgi-bin/mailman/listinfo/cl-store-devel From demmeln at in.tum.de Tue Feb 16 10:07:42 2010 From: demmeln at in.tum.de (Nikolaus Demmel) Date: Tue, 16 Feb 2010 11:07:42 +0100 Subject: [cl-store-devel] CREATE-FLOAT-VALUE defined twice. In-Reply-To: References: <8BB2F7E9-910F-44A3-A863-26CF6D9A3557@in.tum.de> Message-ID: <55F70C49-DEB7-4BC6-8BB2-87C2E6A2544A@in.tum.de> Hi Sean, thanks for your reply. Attached a patch that works for me. It is quite simple. I hope this is acceptable. From the patch description: >Remove warning about CREATE-FLOAT-VALUES being defined twice. > > * I went for the simple fix and just conditionally read the default > version of CREATE-FLOAT-VALUES, which is defined in > default-backend.lisp. For the implementations that provide their > own version of CREATE-FLOAT-VALUE in the according custom.lisp file > this should remove a warning about the function being redefined, > while all other implementations are unaffected by this patch. Best regards, Niko -------------- next part -------------- A non-text attachment was scrubbed... Name: create-float-values-warning.patch Type: application/octet-stream Size: 2825 bytes Desc: not available URL: -------------- next part -------------- Am 16.02.2010 um 10:32 schrieb Sean Ross: > Hi Nikolaus, > A patch which removes the warning would be appreciated. > > Glad you are finding the library useful. > > Cheers, > Sean. > > > On 14 Feb 2010, at 19:30, Nikolaus Demmel wrote: > >> Hi, >> >> the function CREATE-FLOAT-VALUE is defined once in "default-backend.lisp" and once in the implementation specific files. This results in a warning about redefining said function when loading the system. Is there a reason the default definition is there (it does nothing) and would it be possible to remove it and just leave the note saying that each implementation must provide it? I'd be happy to provide a patch. >> >> Btw thanks for this great library! I found it to be very usefull (especially the extensibility and the circularity handling). >> >> Regards, >> Niko_______________________________________________ >> cl-store-devel mailing list >> cl-store-devel at common-lisp.net >> http://common-lisp.net/cgi-bin/mailman/listinfo/cl-store-devel > -------------- next part -------------- A non-text attachment was scrubbed... Name: smime.p7s Type: application/pkcs7-signature Size: 4623 bytes Desc: not available URL: From rosssd at gmail.com Tue Feb 16 10:57:03 2010 From: rosssd at gmail.com (Sean Ross) Date: Tue, 16 Feb 2010 10:57:03 +0000 Subject: [cl-store-devel] CREATE-FLOAT-VALUE defined twice. In-Reply-To: <55F70C49-DEB7-4BC6-8BB2-87C2E6A2544A@in.tum.de> References: <8BB2F7E9-910F-44A3-A863-26CF6D9A3557@in.tum.de> <55F70C49-DEB7-4BC6-8BB2-87C2E6A2544A@in.tum.de> Message-ID: Thanks, I've applied the patch to the repository. - sean On 16 Feb 2010, at 10:07, Nikolaus Demmel wrote: > Hi Sean, > > thanks for your reply. > > Attached a patch that works for me. It is quite simple. I hope this is acceptable. > > From the patch description: > >> Remove warning about CREATE-FLOAT-VALUES being defined twice. >> >> * I went for the simple fix and just conditionally read the default >> version of CREATE-FLOAT-VALUES, which is defined in >> default-backend.lisp. For the implementations that provide their >> own version of CREATE-FLOAT-VALUE in the according custom.lisp file >> this should remove a warning about the function being redefined, >> while all other implementations are unaffected by this patch. > > Best regards, > Niko > > > > > Am 16.02.2010 um 10:32 schrieb Sean Ross: > >> Hi Nikolaus, >> A patch which removes the warning would be appreciated. >> >> Glad you are finding the library useful. >> >> Cheers, >> Sean. >> >> >> On 14 Feb 2010, at 19:30, Nikolaus Demmel wrote: >> >>> Hi, >>> >>> the function CREATE-FLOAT-VALUE is defined once in "default-backend.lisp" and once in the implementation specific files. This results in a warning about redefining said function when loading the system. Is there a reason the default definition is there (it does nothing) and would it be possible to remove it and just leave the note saying that each implementation must provide it? I'd be happy to provide a patch. >>> >>> Btw thanks for this great library! I found it to be very usefull (especially the extensibility and the circularity handling). >>> >>> Regards, >>> Niko_______________________________________________ >>> cl-store-devel mailing list >>> cl-store-devel at common-lisp.net >>> http://common-lisp.net/cgi-bin/mailman/listinfo/cl-store-devel >> > From demmeln at in.tum.de Tue Feb 16 12:06:38 2010 From: demmeln at in.tum.de (Nikolaus Demmel) Date: Tue, 16 Feb 2010 13:06:38 +0100 Subject: [cl-store-devel] CREATE-FLOAT-VALUE defined twice. In-Reply-To: References: <8BB2F7E9-910F-44A3-A863-26CF6D9A3557@in.tum.de> <55F70C49-DEB7-4BC6-8BB2-87C2E6A2544A@in.tum.de> Message-ID: <6A5107D8-CF18-4411-8789-2C00DF5DDBB2@in.tum.de> Thanks! Am 16.02.2010 um 11:57 schrieb Sean Ross: > Thanks, > I've applied the patch to the repository. > > - sean > On 16 Feb 2010, at 10:07, Nikolaus Demmel wrote: > >> Hi Sean, >> >> thanks for your reply. >> >> Attached a patch that works for me. It is quite simple. I hope this is acceptable. >> >> From the patch description: >> >>> Remove warning about CREATE-FLOAT-VALUES being defined twice. >>> >>> * I went for the simple fix and just conditionally read the default >>> version of CREATE-FLOAT-VALUES, which is defined in >>> default-backend.lisp. For the implementations that provide their >>> own version of CREATE-FLOAT-VALUE in the according custom.lisp file >>> this should remove a warning about the function being redefined, >>> while all other implementations are unaffected by this patch. >> >> Best regards, >> Niko >> >> >> >> >> Am 16.02.2010 um 10:32 schrieb Sean Ross: >> >>> Hi Nikolaus, >>> A patch which removes the warning would be appreciated. >>> >>> Glad you are finding the library useful. >>> >>> Cheers, >>> Sean. >>> >>> >>> On 14 Feb 2010, at 19:30, Nikolaus Demmel wrote: >>> >>>> Hi, >>>> >>>> the function CREATE-FLOAT-VALUE is defined once in "default-backend.lisp" and once in the implementation specific files. This results in a warning about redefining said function when loading the system. Is there a reason the default definition is there (it does nothing) and would it be possible to remove it and just leave the note saying that each implementation must provide it? I'd be happy to provide a patch. >>>> >>>> Btw thanks for this great library! I found it to be very usefull (especially the extensibility and the circularity handling). >>>> >>>> Regards, >>>> Niko_______________________________________________ >>>> cl-store-devel mailing list >>>> cl-store-devel at common-lisp.net >>>> http://common-lisp.net/cgi-bin/mailman/listinfo/cl-store-devel >>> >> > -------------- next part -------------- A non-text attachment was scrubbed... Name: smime.p7s Type: application/pkcs7-signature Size: 4623 bytes Desc: not available URL: