Opened 11 years ago

Closed 11 years ago

#241 closed enhancement (fixed)

Add tests for arumentlist to be present

Reported by: ph3-der-loewe Owned by:
Priority: wishlist Milestone: RPLD major release 0.1.0
Component: RoarAudio PlayList Daemon Version: 0.1.0
Keywords: Cc:
Architecture: Compiler:
Difficulty: easy Kernel:
Operating System: Parent Tickets:
Patch attached: no Protocol:
Sound driver: Topic: Hardening

Description

The simple protocol driver module should check if the argument list is present in all commands using it. This should be done using an universal macro so there is only one location of possible errors.

This is strictly speaking not needed, it should just avoid errors in per-command code not checking correctly.

A protocol error should be returned in case of a missing argument list on commands needing one.

Subtickets

Change History (2)

comment:1 Changed 11 years ago by ph3-der-loewe

  • Milestone set to RPLD major release 0.1.0

comment:2 Changed 11 years ago by ph3-der-loewe

  • Protocol RPLD deleted
  • Resolution set to fixed
  • Status changed from new to closed
  • Version changed from current to 0.1rc8

This has been fixed in both the rpld and the mpd protocol.

Note: See TracTickets for help on using tickets.