andrewrk / andrewrk/libsoundio

Learn about tmp directory from the TMPDIR environment variable

Aperta
#128 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement
Lingua principale
C
Stelle
2.1k
Fork
254
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

In the OS abstraction layer, we have this code:

```
char shm_path[] = "/dev/shm/soundio-XXXXXX";
char tmp_path[] = "/tmp/soundio-XXXXXX";
char *chosen_path;

int fd = mkstemp(shm_path);
if (fd < 0) {
fd = mkstemp(tmp_path);
if (fd < 0) {
return SoundIoErrorSystemResources;
} else {
chosen_path = tmp_path;
}
} else {
chosen_path = shm_path;
}
```

I think the part with `/dev/shm/*` is OK, and even the fallback to `/tmp/*` is OK. But let's also check TMPDIR. So it would be 1) try /dev/shm 2) try TMPDIR 3) try /tmp/*.

This would simplify part of #127

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.