Re: bizarre byte-compile issue, possibly due to EIEIO

[Ted Zlatanov <tzz@lifelogs.com>]

On Mon, 04 Apr 2011 17:34:10 +0200 David Engster <deng@randomsample.de> wrote: 

DE> I wasn't thinking. The constructor has to check if there was an initarg
DE> provided for 'data', otherwise it will overwrite it. Hence you have to
DE> use something like

DE>  (defmethod initialize-instance :after ((this registry-db) slots)
DE>    "Set value of data slot of THIS after initialization."
DE>    ;; 'data' will already be set if read from file, so don't overwrite it.
DE>    (with-slots (data tracker) this
DE>      (unless (member :data slots)
DE>        (setq data (make-hash-table :size 10000 :rehash-size 2.0 :test 'equal)))
DE>      (unless (member :tracker slots)
DE>        (setq tracker (make-hash-table :size 100 :rehash-size 2.0)))))

DE> This is also why you shouldn't set the other slot values here.

DE> BTW, you can use 

DE>             :type hash-table

DE> for 'tracker' and 'data'. It will then use 'hash-table-p' for checking
DE> the provided argument.

I took your advice on both the constructor and the type.  The updates
plus some bug fixes are pushed.  I'll test it a bit and if it works OK
will merge the "tzz-gnus-registry-rewrite" branch into "master".

Thanks!
Ted