#apertus IRC Channel Logs

hp_dopeman[m]: really depends on what the reason for the overheadting is

hi

And it seems to me like it might be best to integrate the color and state machine things into the IButton class

and it would allow for a common interface for color setting when building GUIs

battery in the laptop drained, when i was about to answer

am not forcing clean interfaces there, as we are in embedded environment and often things won't change, as hardware stays the same, so feel free to integrate common things into the interface, as long as they're not too big

I guess IButton could be split into a separate Interface and Base class at a later point, if the need arises

even if the task is claimed, but no updates are there, then you can as in the main channel and if nobody bothers then just claim it

forget it, have seen that you've already applied that ;)

Hi

hi satacker[m]

is the challenge also solved already?

had*

morning folks!

Hi

hello

Okay, that'll take some time. Thanks for clarification.

i.e. instead of having gpio42, this would present the respective gpio as something like i2c_nw_power_enable

It will take time for me to go through upstream repository and changelog

i.e. how and where to set the names, how to handle 'discovery' etc

Bertl : I've tried to fix the indentations and the whitespaces..is the format correct now?

let me check ...

port( <- missing space before (

process(SCK) <- similar missing space

in spislave16.vhd there are a huge number of sporadic empty lines?

empty lines make sense to separate blocks, like for example after an if block, before the else

you are also missing a copyright notice and license information

do i need to add the license information in all files ?

really depends on the license, for example for GPL, you want basically a 'one line' entry with your copyright which refers to the license

vnksnkr: just a tip, proper commit messages make it easier to track the history of the project and find certain commits

I should have added that.. thanks for pointing that out..should i add comments instead to the recent commits ?

comments on the commit..isnt there an option to add comments to the already commited ones

sure..will make sure to add commit messages next time