Skip to content

Problem: get_pid returns dummy value on MorphOS and AROS.#8444

Closed
sodero wants to merge 1 commit intovim:masterfrom
sodero:fix/get_pid_dummy_vale_on_aros_and_morphos
Closed

Problem: get_pid returns dummy value on MorphOS and AROS.#8444
sodero wants to merge 1 commit intovim:masterfrom
sodero:fix/get_pid_dummy_vale_on_aros_and_morphos

Conversation

@sodero
Copy link
Contributor

@sodero sodero commented Jun 24, 2021

Solution: Use FindTask to return something which is unique
to all processes.

Solution: Use FindTask to return something which is unique
to all processes.
@codecov
Copy link

codecov bot commented Jun 24, 2021

Codecov Report

Merging #8444 (7a0b46d) into master (d415d26) will decrease coverage by 0.07%.
The diff coverage is n/a.

❗ Current head 7a0b46d differs from pull request most recent head 14c2598. Consider uploading reports for the commit 14c2598 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8444      +/-   ##
==========================================
- Coverage   90.02%   89.94%   -0.08%     
==========================================
  Files         149      149              
  Lines      167830   166231    -1599     
==========================================
- Hits       151090   149519    -1571     
+ Misses      16740    16712      -28     
Flag Coverage Δ
huge-clang-none 88.80% <ø> (-0.37%) ⬇️
huge-gcc-none 89.50% <ø> (-0.01%) ⬇️
huge-gcc-testgui 88.11% <ø> (+<0.01%) ⬆️
huge-gcc-unittests 2.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/sound.c 95.87% <0.00%> (-1.13%) ⬇️
src/help.c 89.76% <0.00%> (-1.08%) ⬇️
src/textobject.c 91.69% <0.00%> (-0.71%) ⬇️
src/drawline.c 84.95% <0.00%> (-0.55%) ⬇️
src/ex_cmds2.c 90.93% <0.00%> (-0.54%) ⬇️
src/beval.c 82.81% <0.00%> (-0.53%) ⬇️
src/netbeans.c 84.04% <0.00%> (-0.52%) ⬇️
src/mbyte.c 80.09% <0.00%> (-0.52%) ⬇️
src/xdiff/xutils.c 78.57% <0.00%> (-0.45%) ⬇️
src/clipboard.c 83.44% <0.00%> (-0.44%) ⬇️
... and 55 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d415d26...14c2598. Read the comment docs.

@brammool brammool closed this in 3a62b14 Jun 24, 2021
janlazo added a commit to janlazo/neovim that referenced this pull request Jul 31, 2021
Problem:    Location list only has the start position.
Solution:   Make it possible to add an end position. (Shane-XB-Qian,
            closes vim/vim#8393)
vim/vim@6864efa

N/A patches for version.c:

vim-patch:8.2.3002: Vim doesn't abort on a fatal Tcl error

Problem:    Vim doesn't abort on a fatal Tcl error.
Solution:   Change emsg() to iemsg(). (Dominique Pellé, closes vim/vim#8383)
vim/vim@affd0bc

vim-patch:8.2.3030: Coverity reports a memory leak

Problem:    Coverity reports a memory leak.
Solution:   Fix the leak and a few typos. (Dominique Pellé, closes vim/vim#8418)
vim/vim@cb54bc6

Patch v8.2.3022 is mostly N/A but cannot be included here
because of new feature check for "has()".

vim-patch:8.2.3032: build problems with MSVC, other crypt issues with libsodium

Problem:    Build problems with MSVC, other crypt issues with libsodium.
Solution:   Adjust MSVC makefile. Disable swap file only when 'key' is set.
            Adjust error message used when key is wrong.  Fix Coverity issues.
            (Christian Brabandt, closes vim/vim#8420, closes vim/vim#8411)
vim/vim@226b28b

vim-patch:8.2.3044: Amiga MorphOS and AROS: process ID is not valid

Problem:    Amiga MorphOS and AROS: process ID is not valid.
Solution:   Use FindTask to return something which is unique to all processes.
            (Ola Söder, closes vim/vim#8444)
vim/vim@3a62b14

vim-patch:8.2.3046: Amiga MorphOS: Term mode is set using DOS packets

Problem:    Amiga MorphOS: Term mode is set using DOS packets.
Solution:   Use the same way of setting term mdoe on all next gen Amiga-like
            systems.  (Ola Söder, closes vim/vim#8445)
vim/vim@b420ac9
chrisbra pushed a commit to chrisbra/vim that referenced this pull request Aug 30, 2021
Problem:    Amiga MorphOS and AROS: process ID is not valid.
Solution:   Use FindTask to return something which is unique to all processes.
            (Ola Söder, closes vim#8444)
lewis6991 pushed a commit to lewis6991/neovim that referenced this pull request Dec 12, 2021
Problem:    Location list only has the start position.
Solution:   Make it possible to add an end position. (Shane-XB-Qian,
            closes vim/vim#8393)
vim/vim@6864efa

N/A patches for version.c:

vim-patch:8.2.3002: Vim doesn't abort on a fatal Tcl error

Problem:    Vim doesn't abort on a fatal Tcl error.
Solution:   Change emsg() to iemsg(). (Dominique Pellé, closes vim/vim#8383)
vim/vim@affd0bc

vim-patch:8.2.3030: Coverity reports a memory leak

Problem:    Coverity reports a memory leak.
Solution:   Fix the leak and a few typos. (Dominique Pellé, closes vim/vim#8418)
vim/vim@cb54bc6

Patch v8.2.3022 is mostly N/A but cannot be included here
because of new feature check for "has()".

vim-patch:8.2.3032: build problems with MSVC, other crypt issues with libsodium

Problem:    Build problems with MSVC, other crypt issues with libsodium.
Solution:   Adjust MSVC makefile. Disable swap file only when 'key' is set.
            Adjust error message used when key is wrong.  Fix Coverity issues.
            (Christian Brabandt, closes vim/vim#8420, closes vim/vim#8411)
vim/vim@226b28b

vim-patch:8.2.3044: Amiga MorphOS and AROS: process ID is not valid

Problem:    Amiga MorphOS and AROS: process ID is not valid.
Solution:   Use FindTask to return something which is unique to all processes.
            (Ola Söder, closes vim/vim#8444)
vim/vim@3a62b14

vim-patch:8.2.3046: Amiga MorphOS: Term mode is set using DOS packets

Problem:    Amiga MorphOS: Term mode is set using DOS packets.
Solution:   Use the same way of setting term mdoe on all next gen Amiga-like
            systems.  (Ola Söder, closes vim/vim#8445)
vim/vim@b420ac9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants