summaryrefslogblamecommitdiffstats
path: root/2021/captions/janitor.md
blob: 3e7bdc3c7c05d4941b97d5f1bfb7839fde218dbb (plain) (tree)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596



















































































































































































































































































































































































































































































































































































































                                                                                                                             
<a name="transcript"></a>
# Transcript

[[!template text="Hello, my name is Stefan Monnier," start="00:00:01.520" video="mainVideo" id=subtitle]]
[[!template text="and I'm going to talk to you about--" start="00:00:04.400" video="mainVideo" id=subtitle]]
[[!template text="well, I'm going to present a bit" start="00:00:06.799" video="mainVideo" id=subtitle]]
[[!template text="of the life of a janitor." start="00:00:08.240" video="mainVideo" id=subtitle]]
[[!template new="1" text="So by and large, there's just" start="00:00:11.840" video="mainVideo" id=subtitle]]
[[!template text="nothing to see here," start="00:00:14.050" video="mainVideo" id=subtitle]]
[[!template text="and that's probably" start="00:00:16.299" video="mainVideo" id=subtitle]]
[[!template text="not super interesting," start="00:00:17.199" video="mainVideo" id=subtitle]]
[[!template text="but some of you might actually like to" start="00:00:18.240" video="mainVideo" id=subtitle]]
[[!template text="see how I work, so I figured why not." start="00:00:19.920" video="mainVideo" id=subtitle]]
[[!template new="1" text="Usually what I do just doesn't make any" start="00:00:25.359" video="mainVideo" id=subtitle]]
[[!template text="any significant difference," start="00:00:27.279" video="mainVideo" id=subtitle]]
[[!template text="and so I basically take existing code" start="00:00:29.920" video="mainVideo" id=subtitle]]
[[!template text="that's working, and I try to change it" start="00:00:32.160" video="mainVideo" id=subtitle]]
[[!template text="hopefully without breaking it too much" start="00:00:35.040" video="mainVideo" id=subtitle]]
[[!template text="and make it slightly more..." start="00:00:37.680" video="mainVideo" id=subtitle]]
[[!template text="you know, following some of the more" start="00:00:40.079" video="mainVideo" id=subtitle]]
[[!template text="modern style, let's say," start="00:00:42.719" video="mainVideo" id=subtitle]]
[[!template text="and sometimes along the way," start="00:00:44.640" video="mainVideo" id=subtitle]]
[[!template text="it actually fixes some bugs." start="00:00:46.719" video="mainVideo" id=subtitle]]
[[!template new="1" text="More concretely, the kind of things" start="00:00:50.399" video="mainVideo" id=subtitle]]
[[!template text="that I do is basically activate" start="00:00:51.983" video="mainVideo" id=subtitle]]
[[!template text="lexical scoping--" start="00:00:54.079" video="mainVideo" id=subtitle]]
[[!template text="that's really my main goal usually--" start="00:00:54.480" video="mainVideo" id=subtitle]]
[[!template text="but also do things like convert" start="00:00:56.239" video="mainVideo" id=subtitle]]
[[!template text="from `cl` to `cl-lib`," start="00:00:58.960" video="mainVideo" id=subtitle]]
[[!template text="sometimes I have to" start="00:01:00.719" video="mainVideo" id=subtitle]]
[[!template text="fix some compilation dependencies," start="00:01:01.440" video="mainVideo" id=subtitle]]
[[!template text="I might convert from `defadvice` to `advice-add`," start="00:01:03.760" video="mainVideo" id=subtitle]]
[[!template text="and many of the things--" start="00:01:07.280" video="mainVideo" id=subtitle]]
[[!template text="in terms of number of changes," start="00:01:11.439" video="mainVideo" id=subtitle]]
[[!template text="most of them are actually" start="00:01:13.119" video="mainVideo" id=subtitle]]
[[!template text="changing `quote fun` to `hash quote fun`" start="00:01:14.000" video="mainVideo" id=subtitle]]
[[!template text="because I prefer it," start="00:01:16.560" video="mainVideo" id=subtitle]]
[[!template text="but also it often helps me" start="00:01:17.360" video="mainVideo" id=subtitle]]
[[!template text="have a better understanding" start="00:01:19.920" video="mainVideo" id=subtitle]]
[[!template text="of which function is called where," start="00:01:21.439" video="mainVideo" id=subtitle]]
[[!template text="and so the warnings I get from it" start="00:01:23.920" video="mainVideo" id=subtitle]]
[[!template text="sometimes help me.  You look concretely..." start="00:01:26.799" video="mainVideo" id=subtitle]]
[[!template text="it's not nothing really clear;" start="00:01:28.799" video="mainVideo" id=subtitle]]
[[!template text="it's more in terms of helping me" start="00:01:30.880" video="mainVideo" id=subtitle]]
[[!template text="have a mental image" start="00:01:33.360" video="mainVideo" id=subtitle]]
[[!template text="of how the package works." start="00:01:35.759" video="mainVideo" id=subtitle]]
[[!template new="1" text="So let's take a look." start="00:01:39.439" video="mainVideo" id=subtitle]]
[[!template text="I'm going to start with" start="00:01:42.880" video="mainVideo" id=subtitle]]
[[!template text="the package `heap`," start="00:01:45.840" video="mainVideo" id=subtitle]]
[[!template text="which I saw had a few weird things in it," start="00:01:46.799" video="mainVideo" id=subtitle]]
[[!template text="so I'm going to compile it." start="00:01:50.560" video="mainVideo" id=subtitle]]
[[!template text="That's basically the way the way I work," start="00:01:53.680" video="mainVideo" id=subtitle]]
[[!template text="right. I take a package." start="00:01:55.600" video="mainVideo" id=subtitle]]
[[!template text="I just pass it to the byte compiler." start="00:01:57.840" video="mainVideo" id=subtitle]]
[[!template text="I do that by just having" start="00:02:00.479" video="mainVideo" id=subtitle]]
[[!template text="a clone of the whole" start="00:02:02.159" video="mainVideo" id=subtitle]]
[[!template text="GNU ELPA repository," start="00:02:04.560" video="mainVideo" id=subtitle]]
[[!template text="and so that's why I built them." start="00:02:06.799" video="mainVideo" id=subtitle]]
[[!template text="I use the build rules" start="00:02:10.000" video="mainVideo" id=subtitle]]
[[!template text="from the GNU ELPA repository." start="00:02:11.520" video="mainVideo" id=subtitle]]
[[!template new="1" text="These build rules enforce--" start="00:02:15.120" video="mainVideo" id=subtitle]]
[[!template text="make sure that the files" start="00:02:16.720" video="mainVideo" id=subtitle]]
[[!template text="are compiled in a clean environment" start="00:02:17.680" video="mainVideo" id=subtitle]]
[[!template text="so you get fairly good warnings." start="00:02:19.680" video="mainVideo" id=subtitle]]
[[!template text="If you look at the warnings you see here," start="00:02:21.920" video="mainVideo" id=subtitle]]
[[!template text="there's a lot of things" start="00:02:23.680" video="mainVideo" id=subtitle]]
[[!template text="which are completely irrelevant," start="00:02:24.720" video="mainVideo" id=subtitle]]
[[!template text="which are due to details" start="00:02:26.480" video="mainVideo" id=subtitle]]
[[!template text="of the way I have my Emacs set up" start="00:02:28.400" video="mainVideo" id=subtitle]]
[[!template text="and some of the local changes" start="00:02:30.319" video="mainVideo" id=subtitle]]
[[!template text="I had in it so, you know," start="00:02:31.599" video="mainVideo" id=subtitle]]
[[!template text="there's no point" start="00:02:34.319" video="mainVideo" id=subtitle]]
[[!template text="paying too much attention to it," start="00:02:35.280" video="mainVideo" id=subtitle]]
[[!template text="but here we have a first warning." start="00:02:37.920" video="mainVideo" id=subtitle]]
[[!template text="We see that this is using `cl`," start="00:02:40.400" video="mainVideo" id=subtitle]]
[[!template text="so we want to change this to `cl-lib`," start="00:02:42.959" video="mainVideo" id=subtitle]]
[[!template text="but that also means" start="00:02:45.040" video="mainVideo" id=subtitle]]
[[!template text="that we may have a new dependency" start="00:02:46.879" video="mainVideo" id=subtitle]]
[[!template text="on the `cl-lib` package," start="00:02:48.400" video="mainVideo" id=subtitle]]
[[!template text="so we have to go check" start="00:02:49.920" video="mainVideo" id=subtitle]]
[[!template text="the start of the file" start="00:02:51.120" video="mainVideo" id=subtitle]]
[[!template text="to see if it already declares" start="00:02:52.000" video="mainVideo" id=subtitle]]
[[!template text="some dependency, and we see it doesn't," start="00:02:54.080" video="mainVideo" id=subtitle]]
[[!template text="not even on a on a recent-enough Emacs," start="00:02:56.800" video="mainVideo" id=subtitle]]
[[!template text="so we have to add--" start="00:03:00.640" video="mainVideo" id=subtitle]]
[[!template text="sorry, that's not going very well..." start="00:03:02.325" video="mainVideo" id=subtitle]]
[[!template text="oh..." start="00:03:05.360" video="mainVideo" id=subtitle]]
[[!template text="okay, we're going to get there somewhere..." start="00:03:06.480" video="mainVideo" id=subtitle]]
[[!template text="somehow..." start="00:03:08.560" video="mainVideo" id=subtitle]]
[[!template text="oh, that still wasn't it, wow, okay--" start="00:03:13.200" video="mainVideo" id=subtitle]]
[[!template text="and along the way..." start="00:03:20.480" video="mainVideo" id=subtitle]]
[[!template new="1" text="Of course, since we converted to `cl-lib`," start="00:03:22.159" video="mainVideo" id=subtitle]]
[[!template text="we have to update the uses" start="00:03:24.159" video="mainVideo" id=subtitle]]
[[!template text="so `defstruct` shouldn't be used anymore." start="00:03:26.159" video="mainVideo" id=subtitle]]
[[!template text="We may want to reindent this" start="00:03:29.840" video="mainVideo" id=subtitle]]
[[!template text="to get something a bit cleaner." start="00:03:31.599" video="mainVideo" id=subtitle]]
[[!template text="We have here a missing quote..." start="00:03:37.040" video="mainVideo" id=subtitle]]
[[!template text="hash, sorry." start="00:03:40.589" video="mainVideo" id=subtitle]]
[[!template text="We have `decf`, so `decf` is here," start="00:03:41.920" video="mainVideo" id=subtitle]]
[[!template text="and that needs to be replaced" start="00:03:46.480" video="mainVideo" id=subtitle]]
[[!template text="with `cl-decf`." start="00:03:48.000" video="mainVideo" id=subtitle]]
[[!template text="Sometimes it's worth doing" start="00:03:49.920" video="mainVideo" id=subtitle]]
[[!template text="a search-and-replace." start="00:03:51.120" video="mainVideo" id=subtitle]]
[[!template text="Here I see there's only two," start="00:03:53.360" video="mainVideo" id=subtitle]]
[[!template text="so it's not worth the trouble;" start="00:03:54.799" video="mainVideo" id=subtitle]]
[[!template text="I just do it by hand, and that's it." start="00:03:57.760" video="mainVideo" id=subtitle]]
[[!template text="Well, that was easy." start="00:04:00.711" video="mainVideo" id=subtitle]]
[[!template text="So let's recompile, see what it says." start="00:04:02.000" video="mainVideo" id=subtitle]]
[[!template text="Ah, this is clean.  Perfect!" start="00:04:10.159" video="mainVideo" id=subtitle]]
[[!template new="1" text="Let's.. we can go see..." start="00:04:12.959" video="mainVideo" id=subtitle]]
[[!template text="There is another one I had." start="00:04:15.000" video="mainVideo" id=subtitle]]
[[!template text="Was it `counsel`, I think.  Yes." start="00:04:17.280" video="mainVideo" id=subtitle]]
[[!template text="So also I saw some funny things" start="00:04:20.239" video="mainVideo" id=subtitle]]
[[!template text="going on here." start="00:04:24.160" video="mainVideo" id=subtitle]]
[[!template text="So I'm going to do" start="00:04:24.320" video="mainVideo" id=subtitle]]
[[!template text="the same procedure as before:" start="00:04:26.479" video="mainVideo" id=subtitle]]
[[!template text="I just compile the file" start="00:04:31.040" video="mainVideo" id=subtitle]]
[[!template text="and look at the warnings." start="00:04:32.800" video="mainVideo" id=subtitle]]
[[!template text="Oh, we have many more here." start="00:04:38.000" video="mainVideo" id=subtitle]]
[[!template text="So let's see..." start="00:04:40.479" video="mainVideo" id=subtitle]]
[[!template text="Okay, so we have missing quotes--" start="00:04:43.120" video="mainVideo" id=subtitle]]
[[!template text="oh, hashes. They're not really missing;" start="00:04:46.504" video="mainVideo" id=subtitle]]
[[!template text="it's just a personal preference." start="00:04:49.240" video="mainVideo" id=subtitle]]
[[!template new="1" text="Oh, here... here's an important one:" start="00:04:54.639" video="mainVideo" id=subtitle]]
[[!template text="so as you know," start="00:04:57.440" video="mainVideo" id=subtitle]]
[[!template text="if you look at the top of the file," start="00:04:59.280" video="mainVideo" id=subtitle]]
[[!template text="you see that here" start="00:05:00.639" video="mainVideo" id=subtitle]]
[[!template text="it says it's using lexical binding," start="00:05:02.240" video="mainVideo" id=subtitle]]
[[!template text="yet it's not fully using lexical binding," start="00:05:04.960" video="mainVideo" id=subtitle]]
[[!template text="because as we just saw," start="00:05:07.120" video="mainVideo" id=subtitle]]
[[!template text="there's a call to the `eval` function" start="00:05:08.960" video="mainVideo" id=subtitle]]
[[!template text="with only one argument," start="00:05:11.039" video="mainVideo" id=subtitle]]
[[!template text="which means the second argument is nil," start="00:05:11.680" video="mainVideo" id=subtitle]]
[[!template text="which means that the expression read" start="00:05:13.280" video="mainVideo" id=subtitle]]
[[!template text="by `read` here is going to be evaluated" start="00:05:16.880" video="mainVideo" id=subtitle]]
[[!template text="using the old dialects," start="00:05:19.520" video="mainVideo" id=subtitle]]
[[!template text="which is only dynamic scoping." start="00:05:22.160" video="mainVideo" id=subtitle]]
[[!template text="So here I like to just change this" start="00:05:24.240" video="mainVideo" id=subtitle]]
[[!template text="to use lexical scoping," start="00:05:25.680" video="mainVideo" id=subtitle]]
[[!template text="which in most cases" start="00:05:26.800" video="mainVideo" id=subtitle]]
[[!template text="just doesn't make any difference." start="00:05:28.080" video="mainVideo" id=subtitle]]
[[!template text="It just makes me feel better." start="00:05:29.680" video="mainVideo" id=subtitle]]
[[!template new="1" text="So there's lots of those hashes" start="00:05:35.919" video="mainVideo" id=subtitle]]
[[!template text="all over the place." start="00:05:40.160" video="mainVideo" id=subtitle]]
[[!template text="It's not strictly necessary, as you know," start="00:05:43.680" video="mainVideo" id=subtitle]]
[[!template text="but I'm just going to add them anyway." start="00:05:45.680" video="mainVideo" id=subtitle]]
[[!template new="1" text="Here we see" start="00:05:52.479" video="mainVideo" id=subtitle]]
[[!template text="it's not going to warn me here" start="00:05:53.199" video="mainVideo" id=subtitle]]
[[!template text="because it doesn't know" start="00:05:54.800" video="mainVideo" id=subtitle]]
[[!template text="that `ivy-make-magic-action`" start="00:05:55.759" video="mainVideo" id=subtitle]]
[[!template text="takes a function," start="00:05:57.600" video="mainVideo" id=subtitle]]
[[!template text="but it's a pretty good guess that it does." start="00:05:58.400" video="mainVideo" id=subtitle]]
[[!template text="And here's some more." start="00:06:12.319" video="mainVideo" id=subtitle]]
[[!template text="What else do we have?" start="00:06:14.479" video="mainVideo" id=subtitle]]
[[!template text="Is that all we have here?" start="00:06:16.080" video="mainVideo" id=subtitle]]
[[!template text="Well, looks like it.  Oh, I see a few..." start="00:06:19.120" video="mainVideo" id=subtitle]]
[[!template text="a few more here..." start="00:06:21.440" video="mainVideo" id=subtitle]]
[[!template text="and one more." start="00:06:27.680" video="mainVideo" id=subtitle]]
[[!template new="1" text="And oh, this is more interesting." start="00:06:30.639" video="mainVideo" id=subtitle]]
[[!template text="So here we have a use of `defadvice`," start="00:06:33.039" video="mainVideo" id=subtitle]]
[[!template text="so if we go back" start="00:06:35.280" video="mainVideo" id=subtitle]]
[[!template text="to the beginning of the file," start="00:06:37.440" video="mainVideo" id=subtitle]]
[[!template text="we see that it actually depends" start="00:06:40.720" video="mainVideo" id=subtitle]]
[[!template text="on Emacs 24.5, so it actually has" start="00:06:42.880" video="mainVideo" id=subtitle]]
[[!template text="the new advice system available" start="00:06:47.360" video="mainVideo" id=subtitle]]
[[!template text="without having to add any dependency," start="00:06:49.280" video="mainVideo" id=subtitle]]
[[!template text="so there's really no good reason" start="00:06:51.520" video="mainVideo" id=subtitle]]
[[!template text="to keep this." start="00:06:53.599" video="mainVideo" id=subtitle]]
[[!template text="So we just convert this" start="00:06:54.880" video="mainVideo" id=subtitle]]
[[!template text="to an `advice-add`," start="00:06:56.160" video="mainVideo" id=subtitle]]
[[!template text="so it just says, you know," start="00:06:58.560" video="mainVideo" id=subtitle]]
[[!template text="this is the function that's advised." start="00:06:59.840" video="mainVideo" id=subtitle]]
[[!template text="This was a `before` advice." start="00:07:02.319" video="mainVideo" id=subtitle]]
[[!template text="The `before` advice, sometimes," start="00:07:04.560" video="mainVideo" id=subtitle]]
[[!template text="when we convert it to `advice-add`," start="00:07:05.500" video="mainVideo" id=subtitle]]
[[!template text="need to be converted to `around` advice." start="00:07:08.479" video="mainVideo" id=subtitle]]
[[!template text="This is when the function" start="00:07:11.199" video="mainVideo" id=subtitle]]
[[!template text="looks or modifies the argument." start="00:07:13.280" video="mainVideo" id=subtitle]]
[[!template text="In this case, if I look at it," start="00:07:15.840" video="mainVideo" id=subtitle]]
[[!template text="I see it doesn't seem to be using" start="00:07:18.639" video="mainVideo" id=subtitle]]
[[!template text="the arguments at all." start="00:07:20.319" video="mainVideo" id=subtitle]]
[[!template text="So I'm just going to keep it" start="00:07:21.280" video="mainVideo" id=subtitle]]
[[!template text="as a `before` advice." start="00:07:25.280" video="mainVideo" id=subtitle]]
[[!template text="And we have to give it a name." start="00:07:27.520" video="mainVideo" id=subtitle]]
[[!template text="Well, we don't really have to," start="00:07:28.672" video="mainVideo" id=subtitle]]
[[!template text="but it's convenient to give it a name" start="00:07:30.880" video="mainVideo" id=subtitle]]
[[!template text="to the new function." start="00:07:32.800" video="mainVideo" id=subtitle]]
[[!template text="So here, they actually had" start="00:07:34.800" video="mainVideo" id=subtitle]]
[[!template text="given a name to the advice," start="00:07:36.880" video="mainVideo" id=subtitle]]
[[!template text="so we're going to keep it," start="00:07:38.080" video="mainVideo" id=subtitle]]
[[!template text="and indeed it's the only function." start="00:07:39.599" video="mainVideo" id=subtitle]]
[[!template text="This name is not used as a function," start="00:07:41.440" video="mainVideo" id=subtitle]]
[[!template text="so we can use it" start="00:07:43.360" video="mainVideo" id=subtitle]]
[[!template text="as the name of the function." start="00:07:44.160" video="mainVideo" id=subtitle]]
[[!template text="I'm going to add a dash here" start="00:07:46.960" video="mainVideo" id=subtitle]]
[[!template text="because I think this function" start="00:07:49.039" video="mainVideo" id=subtitle]]
[[!template text="is really fundamentally" start="00:07:51.120" video="mainVideo" id=subtitle]]
[[!template text="an internal function." start="00:07:53.039" video="mainVideo" id=subtitle]]
[[!template text="So here I just said I add the advice," start="00:07:54.639" video="mainVideo" id=subtitle]]
[[!template text="but I still need to actually" start="00:07:56.720" video="mainVideo" id=subtitle]]
[[!template text="define the function." start="00:07:58.000" video="mainVideo" id=subtitle]]
[[!template text="So that's what I do here," start="00:08:02.879" video="mainVideo" id=subtitle]]
[[!template text="and we need here to list the arguments" start="00:08:04.160" video="mainVideo" id=subtitle]]
[[!template text="that are going to be taken." start="00:08:06.500" video="mainVideo" id=subtitle]]
[[!template text="I don't know what these are," start="00:08:08.240" video="mainVideo" id=subtitle]]
[[!template text="but I know we're not using them," start="00:08:09.199" video="mainVideo" id=subtitle]]
[[!template text="so we'll just accept anything," start="00:08:10.960" video="mainVideo" id=subtitle]]
[[!template text="and that will do the trick." start="00:08:13.759" video="mainVideo" id=subtitle]]
[[!template text="It's a future-proof as well," start="00:08:16.560" video="mainVideo" id=subtitle]]
[[!template text="so that should work." start="00:08:19.199" video="mainVideo" id=subtitle]]
[[!template new="1" text="Oh, here we have another, so it's" start="00:08:22.240" video="mainVideo" id=subtitle]]
[[!template text="basically the same story, I think." start="00:08:24.160" video="mainVideo" id=subtitle]]
[[!template text="It's a `before` advice as well." start="00:08:29.919" video="mainVideo" id=subtitle]]
[[!template text="It doesn't seem to be using" start="00:08:31.599" video="mainVideo" id=subtitle]]
[[!template text="the argument at all," start="00:08:32.959" video="mainVideo" id=subtitle]]
[[!template text="and let's see if this name is not taken." start="00:08:35.599" video="mainVideo" id=subtitle]]
[[!template text="Yeah, good, so we can just do the same:" start="00:08:38.596" video="mainVideo" id=subtitle]]
[[!template text="turn this into an `advice-add`..." start="00:08:43.360" video="mainVideo" id=subtitle]]
[[!template text="`before`..." start="00:08:46.880" video="mainVideo" id=subtitle]]
[[!template text="I just add a dash here." start="00:08:53.040" video="mainVideo" id=subtitle]]
[[!template text="And same thing--" start="00:09:02.480" video="mainVideo" id=subtitle]]
[[!template text="a function that just takes..." start="00:09:05.440" video="mainVideo" id=subtitle]]
[[!template text="because I don't know which arguments" start="00:09:06.959" video="mainVideo" id=subtitle]]
[[!template text="these are so..." start="00:09:08.240" video="mainVideo" id=subtitle]]
[[!template text="I think that should do the trick." start="00:09:10.480" video="mainVideo" id=subtitle]]
[[!template text="Actually, we see that this function" start="00:09:14.640" video="mainVideo" id=subtitle]]
[[!template text="is very similar to the other one." start="00:09:16.080" video="mainVideo" id=subtitle]]
[[!template text="Let's look at the two side-by-side..." start="00:09:18.560" video="mainVideo" id=subtitle]]
[[!template text="...it really is--" start="00:09:31.519" video="mainVideo" id=subtitle]]
[[!template text="oh, it's not exactly identical..." start="00:09:33.055" video="mainVideo" id=subtitle]]
[[!template text="it's, you know, we could try" start="00:09:36.097" video="mainVideo" id=subtitle]]
[[!template text="to merge them into a single function," start="00:09:39.120" video="mainVideo" id=subtitle]]
[[!template text="but it's probably not worth the trouble" start="00:09:41.680" video="mainVideo" id=subtitle]]
[[!template text="so we can keep it this way." start="00:09:43.279" video="mainVideo" id=subtitle]]
[[!template new="1" text="Okay, next warning: an `eval` again," start="00:09:45.920" video="mainVideo" id=subtitle]]
[[!template text="so I could just add `t` here," start="00:09:48.720" video="mainVideo" id=subtitle]]
[[!template text="but if you look at it a bit more," start="00:09:50.640" video="mainVideo" id=subtitle]]
[[!template text="you see that the code" start="00:09:55.120" video="mainVideo" id=subtitle]]
[[!template text="we're going to evaluate" start="00:09:56.000" video="mainVideo" id=subtitle]]
[[!template text="using either lexical scoping" start="00:09:57.760" video="mainVideo" id=subtitle]]
[[!template text="or dynamic scoping" start="00:09:59.279" video="mainVideo" id=subtitle]]
[[!template text="is actually just evaluating a symbol," start="00:10:00.560" video="mainVideo" id=subtitle]]
[[!template text="since we just call an `intern` here." start="00:10:03.440" video="mainVideo" id=subtitle]]
[[!template text="So instead of replacing this" start="00:10:06.240" video="mainVideo" id=subtitle]]
[[!template text="by adding an argument," start="00:10:07.839" video="mainVideo" id=subtitle]]
[[!template text="I'm just going to call `symbol-value`" start="00:10:09.279" video="mainVideo" id=subtitle]]
[[!template text="because that's exactly" start="00:10:11.680" video="mainVideo" id=subtitle]]
[[!template text="what we need to do here, right." start="00:10:12.640" video="mainVideo" id=subtitle]]
[[!template text="I call this &quot;strength reduction,&quot;" start="00:10:14.480" video="mainVideo" id=subtitle]]
[[!template text="and I'm using" start="00:10:16.320" video="mainVideo" id=subtitle]]
[[!template text="a more primitive function instead," start="00:10:17.200" video="mainVideo" id=subtitle]]
[[!template text="which does just what we need," start="00:10:19.680" video="mainVideo" id=subtitle]]
[[!template text="and this one knows that it has to be" start="00:10:23.200" video="mainVideo" id=subtitle]]
[[!template text="accessed by dynamic scoping, of course." start="00:10:25.680" video="mainVideo" id=subtitle]]
[[!template new="1" text="Here I have a `kmacro-ring`," start="00:10:30.640" video="mainVideo" id=subtitle]]
[[!template text="so here I have a function that uses--" start="00:10:32.959" video="mainVideo" id=subtitle]]
[[!template text="`kmacro-ring` comes from" start="00:10:35.600" video="mainVideo" id=subtitle]]
[[!template text="the `kmacro` package, obviously," start="00:10:37.360" video="mainVideo" id=subtitle]]
[[!template text="and we probably don't want to" start="00:10:39.760" video="mainVideo" id=subtitle]]
[[!template text="`require` `kmacro` package" start="00:10:41.600" video="mainVideo" id=subtitle]]
[[!template text="all over the place in `counsel` itself," start="00:10:42.959" video="mainVideo" id=subtitle]]
[[!template text="because `counsel` can be used" start="00:10:48.560" video="mainVideo" id=subtitle]]
[[!template text="without `kmacro`." start="00:10:50.240" video="mainVideo" id=subtitle]]
[[!template text="So I think we're just going to add" start="00:10:53.279" video="mainVideo" id=subtitle]]
[[!template text="a `defvar` to silence the warning." start="00:10:55.200" video="mainVideo" id=subtitle]]
[[!template text="And we have several more. So we have" start="00:11:05.519" video="mainVideo" id=subtitle]]
[[!template text="`initial-counter-value`. (Sorry.)" start="00:11:10.720" video="mainVideo" id=subtitle]]
[[!template text="We have `kmacro-counter`." start="00:11:20.480" video="mainVideo" id=subtitle]]
[[!template text="Do we have more?" start="00:11:23.360" video="mainVideo" id=subtitle]]
[[!template text="Oh, yes, we do." start="00:11:25.760" video="mainVideo" id=subtitle]]
[[!template text="We have `kmacro-counter-value-start`" start="00:11:28.560" video="mainVideo" id=subtitle]]
[[!template text="and `kmacro-counter-format-start`." start="00:11:35.040" video="mainVideo" id=subtitle]]
[[!template text="Okay." start="00:11:40.839" video="mainVideo" id=subtitle]]
[[!template text="I hope this is it." start="00:11:45.040" video="mainVideo" id=subtitle]]
[[!template text="`kmacro-ring`, `counter`, `ring`..." start="00:11:50.160" video="mainVideo" id=subtitle]]
[[!template text="blah blah blah." start="00:11:52.880" video="mainVideo" id=subtitle]]
[[!template text="Here we have another one, `quote`." start="00:11:54.959" video="mainVideo" id=subtitle]]
[[!template text="Here we have another hash missing." start="00:12:00.240" video="mainVideo" id=subtitle]]
[[!template text="It's not missing..." start="00:12:03.279" video="mainVideo" id=subtitle]]
[[!template text="but same thing here." start="00:12:06.079" video="mainVideo" id=subtitle]]
[[!template text="Okay, this is a function from `kmacro`." start="00:12:12.079" video="mainVideo" id=subtitle]]
[[!template text="We could declare it" start="00:12:16.560" video="mainVideo" id=subtitle]]
[[!template text="just to silence the warning" start="00:12:18.079" video="mainVideo" id=subtitle]]
[[!template text="although we don't actually..." start="00:12:20.880" video="mainVideo" id=subtitle]]
[[!template text="normally, when we declare such things--" start="00:12:22.320" video="mainVideo" id=subtitle]]
[[!template text="same thing with variables--" start="00:12:24.480" video="mainVideo" id=subtitle]]
[[!template text="we should try to make sure that indeed" start="00:12:25.279" video="mainVideo" id=subtitle]]
[[!template text="by the time the code is executed," start="00:12:27.300" video="mainVideo" id=subtitle]]
[[!template text="the function will be available," start="00:12:28.760" video="mainVideo" id=subtitle]]
[[!template text="and then very often is" start="00:12:30.800" video="mainVideo" id=subtitle]]
[[!template text="because there's a `require`" start="00:12:32.800" video="mainVideo" id=subtitle]]
[[!template text="sometimes inside a function," start="00:12:34.320" video="mainVideo" id=subtitle]]
[[!template text="and so we should put" start="00:12:35.680" video="mainVideo" id=subtitle]]
[[!template text="the `declare` function" start="00:12:36.399" video="mainVideo" id=subtitle]]
[[!template text="right after the `require`," start="00:12:37.680" video="mainVideo" id=subtitle]]
[[!template text="but I don't think it's the case here." start="00:12:39.920" video="mainVideo" id=subtitle]]
[[!template text="So I'm just going to to add this." start="00:12:41.839" video="mainVideo" id=subtitle]]
[[!template text="I know this comes from `kmacro`," start="00:12:46.399" video="mainVideo" id=subtitle]]
[[!template text="and I could actually check the arguments." start="00:12:49.040" video="mainVideo" id=subtitle]]
[[!template text="It's just taking an optional argument" start="00:12:56.320" video="mainVideo" id=subtitle]]
[[!template text="so I'm going to put it there," start="00:12:58.480" video="mainVideo" id=subtitle]]
[[!template text="so we have it complete." start="00:13:00.880" video="mainVideo" id=subtitle]]
[[!template new="1" text="Okay, we can just recompile," start="00:13:06.720" video="mainVideo" id=subtitle]]
[[!template text="see what is left" start="00:13:10.800" video="mainVideo" id=subtitle]]
[[!template text="from those warnings we've fixed," start="00:13:14.800" video="mainVideo" id=subtitle]]
[[!template text="and we may have new warnings, in any case," start="00:13:17.760" video="mainVideo" id=subtitle]]
[[!template text="because especially when we add the hashes," start="00:13:21.360" video="mainVideo" id=subtitle]]
[[!template text="it tends to give us more warnings." start="00:13:25.440" video="mainVideo" id=subtitle]]
[[!template text="So we have two more functions" start="00:13:29.519" video="mainVideo" id=subtitle]]
[[!template text="which are not known." start="00:13:31.200" video="mainVideo" id=subtitle]]
[[!template text="You can just add them here..." start="00:13:34.560" video="mainVideo" id=subtitle]]
[[!template text="`set-format &quot;kmacro&quot;`" start="00:13:39.440" video="mainVideo" id=subtitle]]
[[!template text="and same thing for `set-counter`." start="00:13:44.720" video="mainVideo" id=subtitle]]
[[!template text="Okay, whatever." start="00:13:48.160" video="mainVideo" id=subtitle]]
[[!template text="This just takes a `format` argument," start="00:13:54.959" video="mainVideo" id=subtitle]]
[[!template text="and this one just takes an `arg` argument." start="00:13:57.120" video="mainVideo" id=subtitle]]
[[!template text="Okay, so let's see what this says now." start="00:14:05.920" video="mainVideo" id=subtitle]]
[[!template text="Hopefully, there's no warnings anymore." start="00:14:10.800" video="mainVideo" id=subtitle]]
[[!template text="We're done.  Okay!" start="00:14:15.519" video="mainVideo" id=subtitle]]
[[!template new="1" text="Okay, the last one we're going to see" start="00:14:17.839" video="mainVideo" id=subtitle]]
[[!template text="is in `enwc`, I saw the other day..." start="00:14:20.079" video="mainVideo" id=subtitle]]
[[!template text="I think I have it here..." start="00:14:23.440" video="mainVideo" id=subtitle]]
[[!template text="here we go, yes..." start="00:14:27.760" video="mainVideo" id=subtitle]]
[[!template text="so `enwc` is an interesting package here" start="00:14:29.680" video="mainVideo" id=subtitle]]
[[!template text="because it has-- as you can see it has--" start="00:14:32.800" video="mainVideo" id=subtitle]]
[[!template text="it's lexical binding," start="00:14:35.680" video="mainVideo" id=subtitle]]
[[!template text="but actually some of the files in it" start="00:14:37.760" video="mainVideo" id=subtitle]]
[[!template text="do not use lexical binding," start="00:14:39.760" video="mainVideo" id=subtitle]]
[[!template text="so it has been partly converted" start="00:14:42.320" video="mainVideo" id=subtitle]]
[[!template text="but not completely." start="00:14:44.320" video="mainVideo" id=subtitle]]
[[!template text="So here I'm going to" start="00:14:46.160" video="mainVideo" id=subtitle]]
[[!template text="enable lexical binding." start="00:14:49.920" video="mainVideo" id=subtitle]]
[[!template text="I have also, I think, in `cm`..." start="00:14:54.160" video="mainVideo" id=subtitle]]
[[!template text="yes..." start="00:14:58.880" video="mainVideo" id=subtitle]]
[[!template text="so I enable it here," start="00:15:01.199" video="mainVideo" id=subtitle]]
[[!template text="and also, I think, `test`." start="00:15:04.000" video="mainVideo" id=subtitle]]
[[!template new="1" text="The test files are often" start="00:15:07.360" video="mainVideo" id=subtitle]]
[[!template text="somewhat problematic" start="00:15:09.360" video="mainVideo" id=subtitle]]
[[!template text="because very often they're not quite" start="00:15:11.839" video="mainVideo" id=subtitle]]
[[!template text="as heavily tested themselves, actually," start="00:15:15.199" video="mainVideo" id=subtitle]]
[[!template text="or they only run" start="00:15:18.880" video="mainVideo" id=subtitle]]
[[!template text="in very specific contexts," start="00:15:20.320" video="mainVideo" id=subtitle]]
[[!template text="and so they may have problems" start="00:15:22.160" video="mainVideo" id=subtitle]]
[[!template text="with missing `requires` or using packages" start="00:15:24.399" video="mainVideo" id=subtitle]]
[[!template text="which are not explicitly in the dependencies" start="00:15:27.360" video="mainVideo" id=subtitle]]
[[!template text="and those kinds of things." start="00:15:29.199" video="mainVideo" id=subtitle]]
[[!template text="I think this is not the case here," start="00:15:31.279" video="mainVideo" id=subtitle]]
[[!template text="but we'll see." start="00:15:33.360" video="mainVideo" id=subtitle]]
[[!template text="`enwc`..." start="00:15:35.440" video="mainVideo" id=subtitle]]
[[!template text="Yes, I want to save this one and that one." start="00:15:38.880" video="mainVideo" id=subtitle]]
[[!template new="1" text="Let's see what it says." start="00:15:42.320" video="mainVideo" id=subtitle]]
[[!template text="Okay, unused lexical variable `x`..." start="00:15:47.199" video="mainVideo" id=subtitle]]
[[!template text="`x`..." start="00:15:51.440" video="mainVideo" id=subtitle]]
[[!template text="Yes, so here we have an unused variable," start="00:15:52.240" video="mainVideo" id=subtitle]]
[[!template text="and indeed, it's not used." start="00:15:57.120" video="mainVideo" id=subtitle]]
[[!template text="It probably had to be named before" start="00:15:58.320" video="mainVideo" id=subtitle]]
[[!template text="because it was..." start="00:16:00.880" video="mainVideo" id=subtitle]]
[[!template text="with dynamic scoping," start="00:16:04.079" video="mainVideo" id=subtitle]]
[[!template text="the `dotimes` requires" start="00:16:05.120" video="mainVideo" id=subtitle]]
[[!template text="the variable to be named, actually," start="00:16:06.399" video="mainVideo" id=subtitle]]
[[!template text="because it's used internally somehow," start="00:16:08.160" video="mainVideo" id=subtitle]]
[[!template text="but with lexical scoping," start="00:16:10.399" video="mainVideo" id=subtitle]]
[[!template text="that's not the case," start="00:16:11.600" video="mainVideo" id=subtitle]]
[[!template text="so we can just put an underscore." start="00:16:12.320" video="mainVideo" id=subtitle]]
[[!template new="1" text="I'm going to change this" start="00:16:14.079" video="mainVideo" id=subtitle]]
[[!template text="because I really don't like" start="00:16:15.199" video="mainVideo" id=subtitle]]
[[!template text="this three-part `dotimes`." start="00:16:16.880" video="mainVideo" id=subtitle]]
[[!template text="I prefer to have" start="00:16:19.000" video="mainVideo" id=subtitle]]
[[!template text="the return value at the end." start="00:16:21.360" video="mainVideo" id=subtitle]]
[[!template text="It's sort of stashed hidden in the middle." start="00:16:23.040" video="mainVideo" id=subtitle]]
[[!template text="That's just a personal preference." start="00:16:26.480" video="mainVideo" id=subtitle]]
[[!template new="1" text="Okay, what else... we have a `widget`." start="00:16:29.680" video="mainVideo" id=subtitle]]
[[!template text="Okay, this argument here says that" start="00:16:31.920" video="mainVideo" id=subtitle]]
[[!template text="it's not used, so if we look at..." start="00:16:34.000" video="mainVideo" id=subtitle]]
[[!template text="We were here, right?  Yes. Right here." start="00:16:44.320" video="mainVideo" id=subtitle]]
[[!template text="Indeed, `widget` is really not used." start="00:16:47.040" video="mainVideo" id=subtitle]]
[[!template text="(Sorry.)" start="00:16:50.480" video="mainVideo" id=subtitle]]
[[!template text="Here's what I get for using" start="00:16:53.600" video="mainVideo" id=subtitle]]
[[!template text="a somewhat vanilla configuration of Emacs," start="00:16:55.279" video="mainVideo" id=subtitle]]
[[!template text="compared to the one I use..." start="00:16:58.320" video="mainVideo" id=subtitle]]
[[!template text="the personally tricked one." start="00:17:01.279" video="mainVideo" id=subtitle]]
[[!template text="Actually, I can..." start="00:17:04.000" video="mainVideo" id=subtitle]]
[[!template text="so we can just mark this argument" start="00:17:05.439" video="mainVideo" id=subtitle]]
[[!template text="as unused," start="00:17:07.919" video="mainVideo" id=subtitle]]
[[!template text="and we don't want to remove the argument" start="00:17:09.360" video="mainVideo" id=subtitle]]
[[!template text="probably, or maybe we could;" start="00:17:11.199" video="mainVideo" id=subtitle]]
[[!template text="we could see where the function is used," start="00:17:12.480" video="mainVideo" id=subtitle]]
[[!template text="and here we see that it's passed" start="00:17:15.679" video="mainVideo" id=subtitle]]
[[!template text="to a higher-order function," start="00:17:18.542" video="mainVideo" id=subtitle]]
[[!template text="basically, so it's going to be..." start="00:17:20.959" video="mainVideo" id=subtitle]]
[[!template text="We can't really change" start="00:17:24.480" video="mainVideo" id=subtitle]]
[[!template text="the calling convention" start="00:17:25.360" video="mainVideo" id=subtitle]]
[[!template text="so we have to mark the argument" start="00:17:25.760" video="mainVideo" id=subtitle]]
[[!template text="as being just an unused argument," start="00:17:27.120" video="mainVideo" id=subtitle]]
[[!template text="but we're going to still receive it." start="00:17:29.600" video="mainVideo" id=subtitle]]
[[!template text="And here it says same thing:" start="00:17:34.000" video="mainVideo" id=subtitle]]
[[!template text="that `widget` is not used in this function." start="00:17:35.360" video="mainVideo" id=subtitle]]
[[!template text="Let's take a look at the function." start="00:17:38.240" video="mainVideo" id=subtitle]]
[[!template text="Indeed it seems it's not used," start="00:17:40.000" video="mainVideo" id=subtitle]]
[[!template text="and so we're just going to mark it" start="00:17:42.400" video="mainVideo" id=subtitle]]
[[!template text="as unused." start="00:17:44.000" video="mainVideo" id=subtitle]]
[[!template text="This is the part of the conversion" start="00:17:46.480" video="mainVideo" id=subtitle]]
[[!template text="to lexical scoping" start="00:17:48.320" video="mainVideo" id=subtitle]]
[[!template text="that's somewhat tricky sometimes" start="00:17:49.200" video="mainVideo" id=subtitle]]
[[!template text="because we don't really know" start="00:17:51.280" video="mainVideo" id=subtitle]]
[[!template text="whether this variable should be using" start="00:17:53.760" video="mainVideo" id=subtitle]]
[[!template text="lexical scoping or dynamic scoping." start="00:17:56.240" video="mainVideo" id=subtitle]]
[[!template text="The fact that it's not used" start="00:17:58.559" video="mainVideo" id=subtitle]]
[[!template text="is a hint that there's probably" start="00:18:00.480" video="mainVideo" id=subtitle]]
[[!template text="something going on," start="00:18:02.320" video="mainVideo" id=subtitle]]
[[!template text="so either it's not used" start="00:18:03.679" video="mainVideo" id=subtitle]]
[[!template text="because it should be using" start="00:18:04.960" video="mainVideo" id=subtitle]]
[[!template text="dynamic scoping--" start="00:18:06.400" video="mainVideo" id=subtitle]]
[[!template text="it is going to be used" start="00:18:07.200" video="mainVideo" id=subtitle]]
[[!template text="by some other code somewhere else--" start="00:18:08.080" video="mainVideo" id=subtitle]]
[[!template text="or it's really not used" start="00:18:10.480" video="mainVideo" id=subtitle]]
[[!template text="because it's just not used, right," start="00:18:11.840" video="mainVideo" id=subtitle]]
[[!template text="and so we need to distinguish the two," start="00:18:14.000" video="mainVideo" id=subtitle]]
[[!template text="and for that, I basically use" start="00:18:16.320" video="mainVideo" id=subtitle]]
[[!template text="my own judgment." start="00:18:20.880" video="mainVideo" id=subtitle]]
[[!template text="This is based typically on the fact that" start="00:18:22.240" video="mainVideo" id=subtitle]]
[[!template text="this is just a very short name," start="00:18:24.880" video="mainVideo" id=subtitle]]
[[!template text="and most local identifiers use short names," start="00:18:27.760" video="mainVideo" id=subtitle]]
[[!template text="whereas item values used for dynamic scoping" start="00:18:32.000" video="mainVideo" id=subtitle]]
[[!template text="typically have a package prefix" start="00:18:34.400" video="mainVideo" id=subtitle]]
[[!template text="or something like this." start="00:18:36.720" video="mainVideo" id=subtitle]]
[[!template text="So the fact that it's a short name" start="00:18:37.679" video="mainVideo" id=subtitle]]
[[!template text="gives me a good idea." start="00:18:38.960" video="mainVideo" id=subtitle]]
[[!template text="Here in this case," start="00:18:40.880" video="mainVideo" id=subtitle]]
[[!template text="I actually look at the code," start="00:18:41.520" video="mainVideo" id=subtitle]]
[[!template text="and we see that there's nothing in here" start="00:18:42.640" video="mainVideo" id=subtitle]]
[[!template text="that may actually refer" start="00:18:45.600" video="mainVideo" id=subtitle]]
[[!template text="to this variable `widget`," start="00:18:47.039" video="mainVideo" id=subtitle]]
[[!template text="so I think it's safe," start="00:18:48.080" video="mainVideo" id=subtitle]]
[[!template text="but in the general case," start="00:18:49.280" video="mainVideo" id=subtitle]]
[[!template text="we may look here and be surprised," start="00:18:51.360" video="mainVideo" id=subtitle]]
[[!template text="or, you know, you may call out" start="00:18:54.400" video="mainVideo" id=subtitle]]
[[!template text="the functions which may themselves end up" start="00:18:55.760" video="mainVideo" id=subtitle]]
[[!template text="referring to this variable." start="00:18:58.320" video="mainVideo" id=subtitle]]
[[!template text="So sometimes we need to investigate a" start="00:19:00.080" video="mainVideo" id=subtitle]]
[[!template text="little more." start="00:19:02.640" video="mainVideo" id=subtitle]]
[[!template text="We are most of the time not completely sure" start="00:19:03.840" video="mainVideo" id=subtitle]]
[[!template text="whether the result is correct or not," start="00:19:05.919" video="mainVideo" id=subtitle]]
[[!template text="of course, so the other thing" start="00:19:07.520" video="mainVideo" id=subtitle]]
[[!template text="you may want to check" start="00:19:09.520" video="mainVideo" id=subtitle]]
[[!template text="is also uses of things" start="00:19:10.640" video="mainVideo" id=subtitle]]
[[!template text="like `eval` or `symbol-value`." start="00:19:12.160" video="mainVideo" id=subtitle]]
[[!template text="So it's often a good idea to search," start="00:19:14.400" video="mainVideo" id=subtitle]]
[[!template text="and you do a search of `eval`," start="00:19:17.200" video="mainVideo" id=subtitle]]
[[!template text="and you see here it's using `eval`." start="00:19:18.799" video="mainVideo" id=subtitle]]
[[!template text="Hmmm... Okay, so what does this `eval` do?" start="00:19:21.490" video="mainVideo" id=subtitle]]
[[!template text="It's evaluating expressions" start="00:19:24.160" video="mainVideo" id=subtitle]]
[[!template text="that appear in `args` here" start="00:19:25.760" video="mainVideo" id=subtitle]]
[[!template text="so you can see where those args come from," start="00:19:28.240" video="mainVideo" id=subtitle]]
[[!template text="and we see here, these are expressions" start="00:19:31.840" video="mainVideo" id=subtitle]]
[[!template text="that don't do anything very special." start="00:19:35.120" video="mainVideo" id=subtitle]]
[[!template text="It's just using `make-supplicant-choice`," start="00:19:36.840" video="mainVideo" id=subtitle]]
[[!template text="and `make-supplicant-choice` itself" start="00:19:41.520" video="mainVideo" id=subtitle]]
[[!template text="just doesn't refer to `widget`, for example," start="00:19:44.960" video="mainVideo" id=subtitle]]
[[!template text="so you know we should be safe," start="00:19:47.120" video="mainVideo" id=subtitle]]
[[!template text="but while I'm here..." start="00:19:50.000" video="mainVideo" id=subtitle]]
[[!template text="okay, well, then we can do that later." start="00:19:52.559" video="mainVideo" id=subtitle]]
[[!template new="1" text="Well, that's actually the next warning," start="00:19:53.840" video="mainVideo" id=subtitle]]
[[!template text="exactly. So here we see that this is" start="00:19:55.679" video="mainVideo" id=subtitle]]
[[!template text="using the dynamically-scoped dialect," start="00:19:58.080" video="mainVideo" id=subtitle]]
[[!template text="so we convert it to lexical-scoped." start="00:20:00.000" video="mainVideo" id=subtitle]]
[[!template text="Of course, this may introduce errors," start="00:20:02.799" video="mainVideo" id=subtitle]]
[[!template text="but we hope it doesn't." start="00:20:04.559" video="mainVideo" id=subtitle]]
[[!template text="And actually, it was a good change here," start="00:20:07.200" video="mainVideo" id=subtitle]]
[[!template text="because if you see again," start="00:20:08.880" video="mainVideo" id=subtitle]]
[[!template text="this actually evals expressions" start="00:20:12.080" video="mainVideo" id=subtitle]]
[[!template text="that appear here in `args`," start="00:20:14.240" video="mainVideo" id=subtitle]]
[[!template text="and so these are expressions" start="00:20:16.159" video="mainVideo" id=subtitle]]
[[!template text="that are passed here." start="00:20:18.480" video="mainVideo" id=subtitle]]
[[!template text="So this expression here used to be" start="00:20:21.039" video="mainVideo" id=subtitle]]
[[!template text="evaluated with dynamic scoping," start="00:20:23.679" video="mainVideo" id=subtitle]]
[[!template text="even though it appears to be normal code" start="00:20:24.480" video="mainVideo" id=subtitle]]
[[!template text="within this file, which says" start="00:20:28.000" video="mainVideo" id=subtitle]]
[[!template text="it's using lexical scoping," start="00:20:29.760" video="mainVideo" id=subtitle]]
[[!template text="and so there are some remnants" start="00:20:32.559" video="mainVideo" id=subtitle]]
[[!template text="of dynamic scoping all over the place" start="00:20:34.400" video="mainVideo" id=subtitle]]
[[!template text="in Emacs still, because we have" start="00:20:36.640" video="mainVideo" id=subtitle]]
[[!template text="those calls of `eval` with a nil argument." start="00:20:37.840" video="mainVideo" id=subtitle]]
[[!template text="Here we have `cons`..." start="00:20:44.880" video="mainVideo" id=subtitle]]
[[!template text="that needs to be `hash quoted`." start="00:20:47.039" video="mainVideo" id=subtitle]]
[[!template new="1" text="Oh, and we have a reference" start="00:20:52.400" video="mainVideo" id=subtitle]]
[[!template text="to this variable `enwc-edit-id'." start="00:20:54.080" video="mainVideo" id=subtitle]]
[[!template text="So this is clearly" start="00:20:56.720" video="mainVideo" id=subtitle]]
[[!template text="a dynamic-scoped variable." start="00:20:57.520" video="mainVideo" id=subtitle]]
[[!template text="We can either add a `defvar`" start="00:21:00.400" video="mainVideo" id=subtitle]]
[[!template text="to silence the warning," start="00:21:02.000" video="mainVideo" id=subtitle]]
[[!template text="or maybe we can `require` the package." start="00:21:03.440" video="mainVideo" id=subtitle]]
[[!template text="The file that defines it..." start="00:21:06.799" video="mainVideo" id=subtitle]]
[[!template text="So let's see where it's defined." start="00:21:14.080" video="mainVideo" id=subtitle]]
[[!template text="Here it's defined in `enwc.el`," start="00:21:17.360" video="mainVideo" id=subtitle]]
[[!template text="so I'm going to try just to add" start="00:21:21.200" video="mainVideo" id=subtitle]]
[[!template text="the dependency." start="00:21:23.440" video="mainVideo" id=subtitle]]
[[!template text="I'm going to `require` here. This is risky." start="00:21:25.039" video="mainVideo" id=subtitle]]
[[!template text="We'll see when we compile a file later," start="00:21:27.840" video="mainVideo" id=subtitle]]
[[!template text="we may get a circular dependency" start="00:21:30.159" video="mainVideo" id=subtitle]]
[[!template text="because of it." start="00:21:32.320" video="mainVideo" id=subtitle]]
[[!template text="If that's the case, we're going to" start="00:21:34.720" video="mainVideo" id=subtitle]]
[[!template text="have to remove this `require`" start="00:21:36.320" video="mainVideo" id=subtitle]]
[[!template text="and instead put `defvar`s." start="00:21:38.320" video="mainVideo" id=subtitle]]
[[!template text="Sometimes it's worth actually" start="00:21:42.000" video="mainVideo" id=subtitle]]
[[!template text="looking further at the various files" start="00:21:42.559" video="mainVideo" id=subtitle]]
[[!template text="to see how to redefine the dependencies" start="00:21:44.640" video="mainVideo" id=subtitle]]
[[!template text="to break those circular dependencies," start="00:21:48.080" video="mainVideo" id=subtitle]]
[[!template text="but it's often not really" start="00:21:49.840" video="mainVideo" id=subtitle]]
[[!template text="worth the trouble." start="00:21:52.320" video="mainVideo" id=subtitle]]
[[!template text="Oh, no, that's not what--" start="00:21:55.679" video="mainVideo" id=subtitle]]
[[!template text="I'm not going to the right place..." start="00:21:58.400" video="mainVideo" id=subtitle]]
[[!template text="Here I was. So here `edit-map`." start="00:22:01.440" video="mainVideo" id=subtitle]]
[[!template text="Well, we can probably..." start="00:22:07.039" video="mainVideo" id=subtitle]]
[[!template text="it may disappear or..." start="00:22:09.760" video="mainVideo" id=subtitle]]
[[!template text="oh, I see." start="00:22:12.159" video="mainVideo" id=subtitle]]
[[!template text="Okay, so this `edit-map` actually is" start="00:22:13.760" video="mainVideo" id=subtitle]]
[[!template text="defined in this very file." start="00:22:16.320" video="mainVideo" id=subtitle]]
[[!template text="It's just that it's defined later." start="00:22:18.559" video="mainVideo" id=subtitle]]
[[!template text="So all we need to do" start="00:22:20.240" video="mainVideo" id=subtitle]]
[[!template text="is to move this definition" start="00:22:21.600" video="mainVideo" id=subtitle]]
[[!template text="to before its first use," start="00:22:24.320" video="mainVideo" id=subtitle]]
[[!template text="since otherwise it's going to be taken" start="00:22:27.200" video="mainVideo" id=subtitle]]
[[!template text="as lexically-scoped, which we don't want." start="00:22:28.960" video="mainVideo" id=subtitle]]
[[!template new="1" text="And while I'm here, I see this `copy-keymap`." start="00:22:33.520" video="mainVideo" id=subtitle]]
[[!template text="I don't like `copy-keymap`," start="00:22:35.360" video="mainVideo" id=subtitle]]
[[!template text="so I'm going to change this" start="00:22:38.400" video="mainVideo" id=subtitle]]
[[!template text="to a normal keymap," start="00:22:40.960" video="mainVideo" id=subtitle]]
[[!template text="and then I'm just going to use" start="00:22:44.080" video="mainVideo" id=subtitle]]
[[!template text="`set-keymap-parent` instead of `copy-keymap`" start="00:22:46.159" video="mainVideo" id=subtitle]]
[[!template text="to get basically the same result," start="00:22:50.080" video="mainVideo" id=subtitle]]
[[!template text="but without having copied anything." start="00:22:51.600" video="mainVideo" id=subtitle]]
[[!template text="And this one will disappear..." start="00:22:55.280" video="mainVideo" id=subtitle]]
[[!template text="this one as well-- or should hopefully," start="00:22:57.760" video="mainVideo" id=subtitle]]
[[!template text="thanks to the `require`." start="00:23:00.240" video="mainVideo" id=subtitle]]
[[!template text="Here we have a `hash` missing," start="00:23:03.360" video="mainVideo" id=subtitle]]
[[!template text="and we have some functions" start="00:23:09.840" video="mainVideo" id=subtitle]]
[[!template text="which are unknown," start="00:23:11.840" video="mainVideo" id=subtitle]]
[[!template text="so let's see..." start="00:23:14.000" video="mainVideo" id=subtitle]]
[[!template text="Where is this function defined?" start="00:23:14.666" video="mainVideo" id=subtitle]]
[[!template text="Nowhere.  Huh, wonderful, okay." start="00:23:18.240" video="mainVideo" id=subtitle]]
[[!template text="So we'll just leave it like it is," start="00:23:21.679" video="mainVideo" id=subtitle]]
[[!template text="and that's going to be" start="00:23:25.200" video="mainVideo" id=subtitle]]
[[!template text="for the author of the package to fix." start="00:23:27.120" video="mainVideo" id=subtitle]]
[[!template text="How about this one?" start="00:23:31.360" video="mainVideo" id=subtitle]]
[[!template text="Oh, okay, so it's defined in `enwc.el`" start="00:23:37.120" video="mainVideo" id=subtitle]]
[[!template text="so presumably," start="00:23:40.240" video="mainVideo" id=subtitle]]
[[!template text="this is going to disappear as well." start="00:23:41.679" video="mainVideo" id=subtitle]]
[[!template text="One more..." start="00:23:50.159" video="mainVideo" id=subtitle]]
[[!template text="Okay, so this one" start="00:23:56.159" video="mainVideo" id=subtitle]]
[[!template text="is just like the previous one." start="00:23:58.640" video="mainVideo" id=subtitle]]
[[!template text="We're going to leave it at that." start="00:23:59.919" video="mainVideo" id=subtitle]]
[[!template text="And this is it!  Huh, wonderful." start="00:24:04.000" video="mainVideo" id=subtitle]]
[[!template text="So let's recompile." start="00:24:06.720" video="mainVideo" id=subtitle]]
[[!template text="Oh, we have a warning for `fin`." start="00:24:16.080" video="mainVideo" id=subtitle]]
[[!template text="This variable seems not to be used" start="00:24:25.679" video="mainVideo" id=subtitle]]
[[!template text="anywhere in the file, so we're just" start="00:24:28.640" video="mainVideo" id=subtitle]]
[[!template text="going to remove it." start="00:24:32.000" video="mainVideo" id=subtitle]]
[[!template text="I leave it there just in case" start="00:24:33.440" video="mainVideo" id=subtitle]]
[[!template text="someone needs later on" start="00:24:34.880" video="mainVideo" id=subtitle]]
[[!template text="to look for a `fin` variable" start="00:24:36.000" video="mainVideo" id=subtitle]]
[[!template text="to see where it used to be." start="00:24:37.679" video="mainVideo" id=subtitle]]
[[!template text="Again, you know, maybe it's actually used..." start="00:24:39.760" video="mainVideo" id=subtitle]]
[[!template text="yeah, dynamic scoping somehow," start="00:24:41.600" video="mainVideo" id=subtitle]]
[[!template text="but given the short name," start="00:24:43.519" video="mainVideo" id=subtitle]]
[[!template text="I presume this is not the case." start="00:24:46.159" video="mainVideo" id=subtitle]]
[[!template text="Here, oh, that's the code removed" start="00:24:48.960" video="mainVideo" id=subtitle]]
[[!template text="that had a hash missing." start="00:24:51.200" video="mainVideo" id=subtitle]]
[[!template text="That's the one that's not defined." start="00:24:52.559" video="mainVideo" id=subtitle]]
[[!template text="This one is not defined," start="00:24:54.159" video="mainVideo" id=subtitle]]
[[!template text="and this is it." start="00:24:56.799" video="mainVideo" id=subtitle]]
[[!template new="1" text="Let's make a last recompilation" start="00:24:58.000" video="mainVideo" id=subtitle]]
[[!template text="to see if we missed yet something else." start="00:25:03.039" video="mainVideo" id=subtitle]]
[[!template text="Nope, and that's it, okay." start="00:25:06.080" video="mainVideo" id=subtitle]]
[[!template text="Well, here we go; we're done." start="00:25:07.919" video="mainVideo" id=subtitle]]
[[!template text="Okay so this was it." start="00:25:11.200" video="mainVideo" id=subtitle]]
[[!template text="You've seen, I think," start="00:25:14.240" video="mainVideo" id=subtitle]]
[[!template text="pretty much examples of all of those," start="00:25:15.440" video="mainVideo" id=subtitle]]
[[!template text="and I hope you enjoyed it." start="00:25:18.000" video="mainVideo" id=subtitle]]
[[!template text="Lessons to take home:" start="00:25:20.960" video="mainVideo" id=subtitle]]
[[!template text="use the byte compiler." start="00:25:22.580" video="mainVideo" id=subtitle]]
[[!template text="You can also use `flymake-mode` instead." start="00:25:23.919" video="mainVideo" id=subtitle]]
[[!template text="I recommend enabling it as much as you can," start="00:25:26.000" video="mainVideo" id=subtitle]]
[[!template text="and head the warnings." start="00:25:31.600" video="mainVideo" id=subtitle]]
[[!template text="Follow the warnings. Try to fix them." start="00:25:33.520" video="mainVideo" id=subtitle]]
[[!template text="If you can fix all of the warnings," start="00:25:35.440" video="mainVideo" id=subtitle]]
[[!template text="it's always much better," start="00:25:37.200" video="mainVideo" id=subtitle]]
[[!template text="because then the new warnings" start="00:25:38.080" video="mainVideo" id=subtitle]]
[[!template text="really show up." start="00:25:39.200" video="mainVideo" id=subtitle]]
[[!template text="And once you've done it, it's really" start="00:25:40.960" video="mainVideo" id=subtitle]]
[[!template text="kind of-- because there's always" start="00:25:42.880" video="mainVideo" id=subtitle]]
[[!template text="new things coming up." start="00:25:44.559" video="mainVideo" id=subtitle]]
[[!template text="And I think this is it." start="00:25:46.799" video="mainVideo" id=subtitle]]
[[!template text="I hope you liked it, and thank you" start="00:25:48.799" video="mainVideo" id=subtitle]]
[[!template text="for attending this presentation. Bye." start="00:25:50.720" video="mainVideo" id=subtitle]]
[[!template text="(captions by Hannah Miller)" start="00:25:56.000" video="mainVideo" id=subtitle]]