Skip to content

Fixed the build warnings and added gitignore. #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 24, 2017
Merged

Fixed the build warnings and added gitignore. #2

merged 1 commit into from
Jul 24, 2017

Conversation

zuyu
Copy link
Contributor

@zuyu zuyu commented Jul 24, 2017

This small PR added the gitignore file, while fixed the building warnings like unused parameters and comparisons between signed and unsigned as well as the trailing whitespaces.

@nihui nihui merged commit 314f551 into Tencent:master Jul 24, 2017
@zuyu zuyu deleted the build-warning-fix branch July 24, 2017 06:24
@zuyu
Copy link
Contributor Author

zuyu commented Jul 24, 2017

@nihui Thank you for merging my PRs.

Btw, next time please try to merge a PR with squash the commits into one commit or rebase the commits individually onto the base branch; thus there will be no another merge commit. See the link for explanation. The default option merge all of the commits into the base branch creates an additional merge commit, resulting the whole commit history non-linear. Cheers!

@nihui
Copy link
Member

nihui commented Jul 24, 2017

^^:)

@brightzhengh brightzhengh mentioned this pull request Jul 27, 2017
Closed
@gneworld gneworld mentioned this pull request Aug 11, 2017
Closed
headupinclouds added a commit to headupinclouds/ncnn that referenced this pull request Apr 8, 2018
@headupinclouds
Merge pull request Tencent#2 from hunter-packages/pr.openmp.flags
53caf1a 
fix cmake openmp usage and travis
@Nanamare Nanamare mentioned this pull request Apr 13, 2018
Closed
@luccao luccao mentioned this pull request Jul 26, 2018
Closed
This was referenced Oct 15, 2018
Closed
Closed
@zyongLiu zyongLiu mentioned this pull request Nov 2, 2018
Closed
@wuyajun0627 wuyajun0627 mentioned this pull request Jan 17, 2019
Closed
@SOLARleisu SOLARleisu mentioned this pull request Oct 14, 2019
Closed
@FB-wh FB-wh mentioned this pull request Mar 20, 2020
Open
@jiyinghui39 jiyinghui39 mentioned this pull request Apr 9, 2020
Closed
@jiefly jiefly mentioned this pull request May 13, 2020
Closed
@403712387 403712387 mentioned this pull request May 15, 2020
Open
@bestpower bestpower mentioned this pull request May 27, 2020
Closed
restyled-io bot pushed a commit that referenced this pull request Jul 27, 2020
@tpoisonooo
Merge pull request #2 from Tencent/master
ee9abe7 
pull from tencent/ncnn
@sbsb99 sbsb99 mentioned this pull request Dec 28, 2020
Open
@whattheweather whattheweather mentioned this pull request May 11, 2021
Open
@yihenxi yihenxi mentioned this pull request May 17, 2021
Open
This was referenced Sep 29, 2021
Closed
Closed
@liuguicen liuguicen mentioned this pull request Oct 29, 2021
Closed
@wm901115nwpu wm901115nwpu mentioned this pull request Dec 15, 2021
Open
@Always-Naive Always-Naive mentioned this pull request Jan 5, 2022
Closed
@Duanzq-GIT Duanzq-GIT mentioned this pull request Mar 4, 2022
Open
@DHY-2020 DHY-2020 mentioned this pull request Apr 29, 2022
Open
@gmagic10 gmagic10 mentioned this pull request May 12, 2022
Open
@wizardk wizardk mentioned this pull request Jun 14, 2022
Closed
theflyingzamboni referenced this pull request in theflyingzamboni/ncnn-vulkan-python Aug 6, 2022
@theflyingzamboni
Attempt joeyballentine#2
ebeb088
@sankexin sankexin mentioned this pull request Aug 9, 2022
Closed
lucasjinreal pushed a commit to lucasjinreal/ncnn that referenced this pull request Sep 25, 2022
@csukuangfj
Merge pull request Tencent#2 from csukuangfj/for-k2
ce304fa 
fix for innectproduct preventing reducing 2-d tensor to 1-d tensor
@fubuzhuirou1 fubuzhuirou1 mentioned this pull request Feb 16, 2023
Closed
@garbe-github-support garbe-github-support mentioned this pull request Jun 28, 2023
Closed
@jcyao jcyao mentioned this pull request Jul 20, 2023
Closed
@jdskddd jdskddd mentioned this pull request Jul 29, 2023
Closed
@ljdang ljdang mentioned this pull request Mar 19, 2024
Open
@FyhSky FyhSky mentioned this pull request Apr 10, 2024
Closed
@ljdang ljdang mentioned this pull request Apr 17, 2024
Closed
@iamfaith iamfaith mentioned this pull request Jul 4, 2024
Closed
@hallowsman hallowsman mentioned this pull request Jul 23, 2024
Open
@jackynew8669 jackynew8669 mentioned this pull request Sep 4, 2024
Open
@runrunrun1994 runrunrun1994 mentioned this pull request Sep 13, 2024
Closed
@WylAndy WylAndy mentioned this pull request Dec 13, 2024
Closed
@MAXliang2 MAXliang2 mentioned this pull request Mar 31, 2025
Open
@sen-O sen-O mentioned this pull request Aug 11, 2025
Open
@BandageWorm BandageWorm mentioned this pull request Aug 20, 2025
Open
@libofei2004 libofei2004 mentioned this pull request Aug 23, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zuyu @nihui