[சனி ஜூன் 15, 2024] Ihor Radchenko wrote:

> Visuwesh <visuweshm@gmail.com> writes:
>
>>> So, you can set DATA to (list (or dump-func 'org-plot/gnuplot-to-data) table num-cols params)
>>
>> Done in the attached patch.
>
> Thanks!
>
>>        ;; Dump table to datafile
>> -      (let ((dump-func (plist-get type :data-dump)))
>> +      (let* ((dump-func (plist-get type :data-dump)))
>
> Why let*?

Oops, leftover change from a previous iteration of the patch.  Now
fixed.

>> +        (setq data-file (org-babel-temp-stable-file
>> +                         (list (or dump-func 'org-plot/gnuplot-to-data)
>> +                               table num-cols params)
>> +                         "org-plot"))
>
> Please add in-code comment explaining why we need `org-babel-temp-stable-file'.

Now done.