Re: CEDET non-regression bug fixes

[David Engster <deng@randomsample.de>]

Stefan Monnier writes:
>> === modified file 'lisp/cedet/semantic/complete.el'
>> --- lisp/cedet/semantic/complete.el	2012-10-12 19:56:34 +0000
>> +++ lisp/cedet/semantic/complete.el	2012-11-04 21:06:33 +0000
>> @@ -1340,7 +1343,10 @@
>  
>>  (defmethod semantic-displayor-scroll-request ((obj semantic-displayor-abstract))
>>    "A request to for the displayor to scroll the completion list (if needed)."
>> -  (scroll-other-window))
>> +  (with-selected-window (get-buffer-window "*Completions*")
>> +    (if (posn-at-point (point-max))
>> +	(goto-char (point-min))
>> +      (scroll-up))))
>
> This will signal an error when *Completions* is not displayed anywhere.

At this stage, there must be a *Completions* buffer, otherwise it's an
error anyway. But regarding the freeze, I will wrap something around
that making sure it exists.

>> -		   (buff (get-file-buffer file))
>> +		   (buff (find-buffer-visiting file))
>
> We need to do something to avoid those problems.  AFAIK there's no
> reason to ever prefer get-file-buffer over find-buffer-visiting, so
> maybe we should simply get rid of get-file-buffer and make it an alias
> of find-buffer-visiting.

Oh, you have my vote for that. But I have a hunch you won't put this
change into the emacs-24 branch, so I read this comment as "Fine, please
commit". ;-)

-David