Jump to content

Recommended Posts

Posted

Hallo zusammen,

 

mir ist aufgefallen, dass es z.B. beim Distance-IR die Methoden gibt:

int distance_ir_get_distance(DistanceIR *distance_ir, uint16_t *ret_distance);
int distance_ir_set_distance_callback_threshold(DistanceIR *distance_ir, char option, int16_t min, int16_t max);

 

wobei get_distance einen uint16 liefert, aber set_threshold einen int16 als Parameter erwartet. Bei anderen Sensoren nimmt die set_threshold doch den gleichen Typ wie die "get_value" Funktion, z. B. beim Moisture-Sensor und Sound-Intensity-Bricklet.

 

Ist das hier ein Copy/Paste-Fehler oder hat das einen bestimmten Hintergrund?

 

Da macht mir etwas Probleme, die Sensoren einheitlich zu verwenden, wenn die Typen nicht stimmig sind.

  • 2 weeks later...
Posted

Das ist ein Copy-Paste Fehler. Der betrifft mehrere Bricklets. Aber der gültige Werte Bereich für min/max ist bei allen nicht beeinträchtigt. So dass wir die Stabilität der API höher gewertet haben als die Korrektheit der Typen.

 

Als ich das noch mal durchgegangen bin ist mir allerdings aufgefallen, dass das für das Analog In Bricklet (2.0) nicht stimmt und nicht der vollständige Messbereich als min/max angegeben werden kann.

 

Daher haben wir uns jetzt entschieden alle betroffenen Bricklets zu korrigieren.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...