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

NPE fix for #43 #94

Merged

NiklasRentzCAU merged 1 commit into master from nre/43

Aug 16, 2021

Member

NiklasRentzCAU commented Jul 28, 2021

Necessary fix to answer my question in issue #43


          klighd: fixed NPE for labels with only a KText and no direct label text

758ab55

NiklasRentzCAU requested a review from sailingKieler

July 28, 2021 08:42

sailingKieler approved these changes

View reviewed changes

Member

sailingKieler left a comment

This change doesn't hurt for sure, so I'm fine. (although I never tested managed labels)

NiklasRentzCAU merged commit 0b15965 into master

NiklasRentzCAU deleted the nre/43 branch

August 16, 2021 07:14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet