Page MenuHomeFreeBSD

pstef (Piotr Stefaniak)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 25 2016, 9:00 PM (379 w, 3 d)

Recent Activity

Sat, Sep 30

pstef added a comment to D42027: netlink: fix accessing freed memory.

I'm not sure but it seems that on the changed lines you replaced space indentation with tabs.

Sat, Sep 30, 3:40 PM

Sun, Sep 24

pstef accepted D41949: mailmap: Add pstef and val.
Sun, Sep 24, 10:05 AM

Sat, Sep 23

pstef committed rGa675eaec5aef: sh: implement PS1 \D to print current time (authored by pstef).
sh: implement PS1 \D to print current time
Sat, Sep 23, 6:29 PM
pstef closed D35840: sh: implement PS1 \D to print current time.
Sat, Sep 23, 6:28 PM

Thu, Sep 14

pstef added inline comments to D41849: virtio: add comment on the spec for vring struct.
Thu, Sep 14, 6:29 AM
pstef added inline comments to D41851: virtio: start simple, document virtio_endian(9).
Thu, Sep 14, 6:28 AM · manpages
pstef added inline comments to D41853: virtio: add introduction to virtio, document virtio.h.
Thu, Sep 14, 6:23 AM
pstef added inline comments to D41852: virtio: add man page for virtqueue(9) functions.
Thu, Sep 14, 6:05 AM

Tue, Sep 12

pstef committed rGc996638497d1: tslog.4: fix typo (authored by pstef).
tslog.4: fix typo
Tue, Sep 12, 7:11 PM
pstef committed rG8e5c53af831b: sh: also auto-complete functions (authored by pstef).
sh: also auto-complete functions
Tue, Sep 12, 4:43 PM
pstef committed rGbec7b9a21979: sh: introduce a function to iterate over all hashed commands (authored by pstef).
sh: introduce a function to iterate over all hashed commands
Tue, Sep 12, 4:43 PM
pstef committed rG0fd450e2895e: sh: tab-complete aliases (authored by pstef).
sh: tab-complete aliases
Tue, Sep 12, 4:43 PM
pstef committed rG2fc4a84ed828: sh: introduce a function to iterate over all aliases (authored by pstef).
sh: introduce a function to iterate over all aliases
Tue, Sep 12, 4:43 PM
pstef committed rG63b6e661d25c: sh: reindent a for loop in parser.c (authored by pstef).
sh: reindent a for loop in parser.c
Tue, Sep 12, 4:43 PM
pstef closed D40619: sh: auto-complete functions and aliases.
Tue, Sep 12, 4:43 PM
pstef closed D37926: sh: reindent a for loop in parser.c.
Tue, Sep 12, 4:43 PM

Aug 25 2023

pstef requested changes to D16132: New cmb(3) library and cmb(1) utility.

As of stable/14 this no longer compiles for me due to

/usr/src/lib/libcmb/cmb.c:854:6: error: invalid argument type 'void' to unary expression
        if (!BN_zero(count))
            ^~~~~~~~~~~~~~~
/usr/src/lib/libcmb/cmb.c:1032:7: error: invalid argument type 'void' to unary expression
                if (!BN_zero(seq))
                    ^~~~~~~~~~~~~
2 errors generated.
Aug 25 2023, 5:58 PM

Aug 24 2023

pstef committed rG5c0a1c15ff8c: ps: add a new option -D to reimplement tree traversal (authored by pstef).
ps: add a new option -D to reimplement tree traversal
Aug 24 2023, 4:38 PM
pstef committed rGf06e32e9b9e4: Revert "ps: extend the non-standard option -d (tree view) to work with -p" (authored by pstef).
Revert "ps: extend the non-standard option -d (tree view) to work with -p"
Aug 24 2023, 4:37 PM
pstef added a reverting change for rGca8c0d5e8110: ps: extend the non-standard option -d (tree view) to work with -p: rGf06e32e9b9e4: Revert "ps: extend the non-standard option -d (tree view) to work with -p".
Aug 24 2023, 4:37 PM
pstef closed D41231: ps: split tree traversal to a separate option.
Aug 24 2023, 4:37 PM

Aug 20 2023

pstef updated the diff for D41231: ps: split tree traversal to a separate option.
  1. -D doesn't imply -d anymore. I found that to be unnecessarily limiting, now users can decide on the visual format.
  2. The usage output and the manual page are updated.
Aug 20 2023, 8:04 AM

Aug 19 2023

pstef updated the diff for D41231: ps: split tree traversal to a separate option.

Add -D both.

Aug 19 2023, 11:05 PM
pstef updated the diff for D41231: ps: split tree traversal to a separate option.

Follow advice from Jamie and use -D up and -D down.

Aug 19 2023, 10:48 PM

Jul 29 2023

pstef added a reverting change for rGca8c0d5e8110: ps: extend the non-standard option -d (tree view) to work with -p: D41231: ps: split tree traversal to a separate option.
Jul 29 2023, 7:11 PM
pstef requested review of D41231: ps: split tree traversal to a separate option.
Jul 29 2023, 7:11 PM

Jul 7 2023

pstef updated the diff for D40619: sh: auto-complete functions and aliases.

Address comments from Jilles.

Jul 7 2023, 6:40 PM

Jun 29 2023

pstef accepted D40788: Restore and document -ps / -nps option..
Jun 29 2023, 1:54 PM
pstef added a comment to D40788: Restore and document -ps / -nps option..

If it's just an effort to synchronize with OpenBSD then I vote against this change. I intentionally removed this option years ago, so I have to assume that the difference here comes from the fact that OpenBSD didn't.
If, however, you want to restore this functionality for whatever reason, I will only make a face and accept it.

Jun 29 2023, 1:32 PM

Jun 20 2023

pstef updated the summary of D40619: sh: auto-complete functions and aliases.
Jun 20 2023, 8:38 AM
pstef requested review of D40619: sh: auto-complete functions and aliases.
Jun 20 2023, 8:36 AM

Jun 19 2023

pstef committed rG14dd001235f9: sh: also auto-complete built-ins (authored by pstef).
sh: also auto-complete built-ins
Jun 19 2023, 5:28 PM
pstef closed D39839: sh: also auto-complete built-ins.
Jun 19 2023, 5:28 PM

May 13 2023

pstef committed rG411942a70eb6: GENERIC: remove a stray space character (authored by pstef).
GENERIC: remove a stray space character
May 13 2023, 8:06 PM
pstef committed rG43489c142e56: sh: improve style (authored by pstef).
sh: improve style
May 13 2023, 8:06 PM

May 3 2023

pstef added a comment to D37627: Disable errors for -Wnonnull for the kernel for GCC 12..

What are the functions? Generally, passing a null pointer to a standard string function invokes UB regardless of the passed size, and some compilers do make optimizations based on this assumption.

May 3 2023, 6:35 AM

Apr 29 2023

pstef updated the diff for D39839: sh: also auto-complete built-ins.

Reduce code duplication.
I was worried about some repetitive code here when adding the second source of commands (built-ins, after files from $PATH), but it became visibly worse when I started working on adding a third source (aliases). So the sooner I factor this out, the cleaner the future commits will be.

Apr 29 2023, 8:48 PM

Apr 28 2023

pstef added inline comments to D39839: sh: also auto-complete built-ins.
Apr 28 2023, 6:45 AM
pstef updated the diff for D39839: sh: also auto-complete built-ins.

Address reviews.

Apr 28 2023, 6:44 AM

Apr 27 2023

pstef added inline comments to D39839: sh: also auto-complete built-ins.
Apr 27 2023, 4:45 PM
pstef updated the diff for D39839: sh: also auto-complete built-ins.

builtincmd doesn't hold C strings, use memcmp().

Apr 27 2023, 6:13 AM

Apr 26 2023

pstef added a comment to D39839: sh: also auto-complete built-ins.

This doesn't follow the Don't Repeat Yourself principle, but currently I don't have a better idea how to express this code.

Apr 26 2023, 8:12 PM
pstef requested review of D39839: sh: also auto-complete built-ins.
Apr 26 2023, 8:11 PM

Feb 8 2023

pstef accepted D38386: Add nproc(1).
Feb 8 2023, 7:40 PM
pstef accepted D38386: Add nproc(1).

My question about fprintf() remains unanswered, but it is not a blocker.

Feb 8 2023, 12:14 PM

Feb 6 2023

pstef accepted D38399: tarfs: Fix typo in man page..
Feb 6 2023, 3:06 PM

Feb 5 2023

pstef added inline comments to D38386: Add nproc(1).
Feb 5 2023, 10:27 AM
pstef added a comment to D38386: Add nproc(1).

TODO: write the manpage

Feb 5 2023, 10:08 AM
pstef added inline comments to D38386: Add nproc(1).
Feb 5 2023, 8:10 AM
pstef added a comment to D38386: Add nproc(1).

Minor style improvements, nitpicking and questions.

Feb 5 2023, 7:53 AM

Jan 20 2023

pstef added a comment to D38047: Multiple fixes to the NSS caching system.

Please provide more diff context (diff -U999999).

Jan 20 2023, 5:00 PM · Src Committers
pstef added inline comments to D38047: Multiple fixes to the NSS caching system.
Jan 20 2023, 4:55 PM · Src Committers

Jan 17 2023

pstef accepted D38097: df: Return non-zero status on write failure..
Jan 17 2023, 3:21 PM

Jan 8 2023

pstef updated the diff for D35840: sh: implement PS1 \D to print current time.

Back out the previous update.
The -1 was there for the i++ that the loop always performs, so it's needed regardless of whether strftime() returned 0 or more.
Split the calculation that was there before into two parts, one of which just decrements i and makes a comment on why it's done.

Jan 8 2023, 12:56 PM
pstef updated the diff for D35840: sh: implement PS1 \D to print current time.

Minutes after posting the previous version, I thought about handling the rare corner case when strftime() returns 0.

Jan 8 2023, 10:46 AM
pstef updated the diff for D35840: sh: implement PS1 \D to print current time.

Simplify the code part a bit.
Also improve comments and some variable names.

Jan 8 2023, 10:35 AM

Jan 2 2023

pstef added inline comments to D37753: Add tarfs, a filesystem backed by tarballs..
Jan 2 2023, 4:06 PM
pstef added a comment to D35840: sh: implement PS1 \D to print current time.

When I was looking at this code today, I thought that the unconditional memccpy() is a bit wasteful in the \D{} case, because the one copied byte is a special case and when it's noticed, the copied byte is overwriten by strcpy() with "%X".
There is little to be gained in either performance or legibility of the code, but when I rewrote this locally to be less wasteful, it didn't look much worse, nor did it look much better.
Opinions?

Jan 2 2023, 12:10 AM

Jan 1 2023

pstef updated the diff for D35840: sh: implement PS1 \D to print current time.

In this iteration I tried to address all new feedback.
I also was annoyed by the current formatting of the file, so this is a diff against D37926 where I reduce indentation.

Jan 1 2023, 7:46 PM
pstef requested review of D37926: sh: reindent a for loop in parser.c.
Jan 1 2023, 7:43 PM

Dec 29 2022

pstef updated the diff for D35840: sh: implement PS1 \D to print current time.

Drop PS4 support and update the manual page changes according to feedback.

Dec 29 2022, 9:57 AM

Dec 24 2022

pstef committed rG36802990d64c: ls(1): add a -v flag to sort naturally (authored by obiwac_gmail.com).
ls(1): add a -v flag to sort naturally
Dec 24 2022, 7:23 AM
pstef committed rG149466a516b2: ping: split the visual part of -f into a new option -. (authored by pstef).
ping: split the visual part of -f into a new option -.
Dec 24 2022, 7:23 AM

Dec 22 2022

pstef added a comment to D37753: Add tarfs, a filesystem backed by tarballs..
Dec 22 2022, 10:13 PM

Dec 20 2022

pstef added a comment to D37753: Add tarfs, a filesystem backed by tarballs..

Hi, thanks for working on this.

Dec 20 2022, 5:04 PM

Dec 16 2022

pstef accepted D37712: daily 150.clean-hoststat: suppress error when using dma.

LGTM FWIW

Dec 16 2022, 2:48 PM
pstef added inline comments to D37712: daily 150.clean-hoststat: suppress error when using dma.
Dec 16 2022, 10:43 AM

Dec 15 2022

pstef added inline comments to D37695: bin/uuidgen: add support for v4 uuids.
Dec 15 2022, 7:10 AM

Dec 14 2022

pstef added a comment to D37695: bin/uuidgen: add support for v4 uuids.

Should this version become the default? On the one hand, it has POLA violation potential, on the other hand, it's what the command does under another system.

Dec 14 2022, 10:38 AM

Dec 13 2022

pstef added inline comments to D37674: add sizeof(7) man page.
Dec 13 2022, 9:30 PM
pstef accepted D37683: sizeof(7): miscellaneous edits.
Dec 13 2022, 9:16 PM
pstef added a comment to D37674: add sizeof(7) man page.

Initial review.

Dec 13 2022, 7:57 AM

Dec 12 2022

pstef accepted D37676: diff: fix side-by-side output with tabbed input.

hw: column width ((h)ard (w)idth?)

Dec 12 2022, 8:20 PM

Nov 24 2022

pstef added a reviewer for D37478: Add support for mounting single files in nullfs: mjg.
Nov 24 2022, 8:12 PM

Nov 4 2022

pstef added a comment to D37263: getty: code cleanup, part 1.

If you intend to do more work on these files, I suggest improving the style before introducing more significant changes. There is a lot of inherited stylistic issues here, unfortunately.
I think you have a typo in the description and you meant "braces" - I meantion it just in case you planned to copy it to the commit message.

Nov 4 2022, 2:02 PM

Nov 2 2022

pstef added inline comments to D37236: gzip: Add support for decompressing zstd files..
Nov 2 2022, 3:23 PM

Oct 31 2022

pstef added a comment to D36860: diff: Import new diff algorithms from libdiff.

Would it be possible to omit the compat subdirectory from the import?

Oct 31 2022, 2:10 PM

Oct 30 2022

pstef closed D36407: ls: `-v` flag.
Oct 30 2022, 11:14 PM
pstef committed rGe2662256cdbc: ls(1): add a -v flag to sort naturally (authored by obiwac_gmail.com).
ls(1): add a -v flag to sort naturally
Oct 30 2022, 11:14 PM
pstef added inline comments to D34747: ls(1): Change -g and make -n imply -l to implement POSIX.
Oct 30 2022, 4:49 PM
pstef added a comment to D35761: libc: expose a primitive version of b64_pton().

@kib, sorry about not providing some aspects of the context of this change, I thought this was a good enough draft to get the first few reviews, but in hindsight it looks just lazy of me.

Oct 30 2022, 12:13 PM

Oct 23 2022

pstef added reviewers for D35761: libc: expose a primitive version of b64_pton(): kib, imp.
Oct 23 2022, 7:32 AM

Oct 21 2022

pstef accepted D37083: top: improve sort field storage/lookup.

But the summary contains the same phrase twice, seems unintentional.

Oct 21 2022, 10:20 PM

Oct 18 2022

pstef added a comment to D37035: mailwrapper: use dma(8) over sendmail(8).

Please remember to tag the commit as Relnotes: yes.

Oct 18 2022, 2:30 PM

Aug 25 2022

pstef accepted D36314: bintrans: replace the quoted printable encoder/decoder.
Aug 25 2022, 6:29 AM

Aug 24 2022

pstef committed rG6480563d0678: sh: accept fc options grouped behind one '-' (authored by pstef).
sh: accept fc options grouped behind one '-'
Aug 24 2022, 6:23 PM
pstef committed rGead0fe475c5e: sh: nullify ENV in tests (authored by pstef).
sh: nullify ENV in tests
Aug 24 2022, 6:22 PM
pstef added inline comments to D36314: bintrans: replace the quoted printable encoder/decoder.
Aug 24 2022, 3:38 PM

Aug 23 2022

pstef added inline comments to D36314: bintrans: replace the quoted printable encoder/decoder.
Aug 23 2022, 4:02 PM

Aug 22 2022

pstef requested review of D36293: base64: implement -i/-n to ignore non-alphabet characters.
Aug 22 2022, 4:12 PM

Aug 21 2022

pstef added a comment to D35807: libc: Add `strverscmp(3)` & `versionsort(3)`.

Where can I find the rules for vertical spacing? I can't find it explicitly in style(9)

Aug 21 2022, 12:19 PM

Aug 20 2022

pstef committed rGa1423456415a: sh: nullify ENV in tests (authored by pstef).
sh: nullify ENV in tests
Aug 20 2022, 11:31 AM
pstef closed D35876: sh: nullify ENV in tests.
Aug 20 2022, 11:31 AM
pstef closed D35836: sh: accept fc options grouped behind one '-'.
Aug 20 2022, 11:31 AM
pstef committed rG755a1be6d015: sh: accept fc options grouped behind one '-' (authored by pstef).
sh: accept fc options grouped behind one '-'
Aug 20 2022, 11:31 AM

Aug 19 2022

pstef added a comment to D35807: libc: Add `strverscmp(3)` & `versionsort(3)`.

I can do it myself this evening, if that doesn't slow you down for anything else. (Aswell as making that post on -hackers that @lwhsu mentioned, if that's still necessary.)

Aug 19 2022, 11:07 AM
pstef added a comment to D35836: sh: accept fc options grouped behind one '-'.

@jilles is this patch in a good shape now?

Aug 19 2022, 10:49 AM
pstef added a comment to D35807: libc: Add `strverscmp(3)` & `versionsort(3)`.

Hey guys, is there anything here that I still need to do?

Aug 19 2022, 10:42 AM

Aug 16 2022

pstef added a comment to D29577: acpi(4): mention NONE as possible setting for hw.acpi.power_button_state.

I already started looking into adding it to prevent my cat from powering off my system

Aug 16 2022, 8:51 PM
pstef committed rGe45f7e92439d: acpi(4): mention NONE as possible setting for hw.acpi.power_button_state (authored by yuripv).
acpi(4): mention NONE as possible setting for hw.acpi.power_button_state
Aug 16 2022, 8:46 PM