Member Avatar for praddy85

Hi,
Im a novice and im implementing a mp3 player. Im using the concept of doubley linked list for moving between songs in the directory... I have enclosed the code also.. the error i get is

next_previous.c:80: warning: assignment makes integer from pointer without a cast
next_previous.c:81: warning: initialization makes integer from pointer without a cast

This seems simple but for some reason im not able to work this out... any help is greatly appreciated . Thanks.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <dirent.h>

int loadMusicCollection(void);
void readMusicCollection(void);
int saveMusicCollection(void);
void populateMusicCollection(void);


#define TRUE 1
#define FALSE 0

struct song{ 
		int number;
		int *pnumber;
		char letter;
		char *pletter;
		struct dirent *file;
		struct song *next;
		struct song *previous;
	   };

char *strsong[6] = {"song1","song2","song3","song4","song5","song6"};


struct song *firstSong = NULL;
struct song *currentSong = NULL;
struct song *nextSong = NULL;
struct song *previousSong = NULL;
struct song *lastSong = NULL;

char *filename = "music.dat";
FILE *datafile = NULL;


int main(int argc, char *argv[])
{
	if (argc > 1) 
	{
		loadMusicCollection();
                readMusicCollection();
	}
	else
	{
		firstSong = malloc(sizeof(*firstSong));
		
		if(firstSong == NULL)
		printf("Error allocating memory\n");
		currentSong = firstSong;	
		lastSong = previousSong;
		lastSong->next = NULL;
		free(currentSong);
		

                populateMusicCollection();
		saveMusicCollection();
                readMusicCollection();
       }
		
		return 0;

}

void populateMusicCollection(void)
{
	
	int i;
	int *tmp;
	int tmpval;

	for (i=0;i<5;i++)
	{
		currentSong->number = i;
		tmpval = i;
		tmp = &tmpval;
		currentSong->pnumber = tmp;
		currentSong->letter = strsong[i];
		char tmpletter = strsong[i];
		currentSong->pletter = &tmpletter;

                printf("Populating, number = %d\n", currentSong->number);
		printf("Populating, pnumber = %d\n", *currentSong->pnumber);
		printf("Populating, letter = %c\n", currentSong->letter);
		printf("Populating, pletter = %c\n", *currentSong->pletter);

		if (previousSong == NULL) 
			currentSong->previous = NULL;
		else
			currentSong->previous = previousSong;

		nextSong = malloc(sizeof(*nextSong));
		if (nextSong == NULL)
			printf("Error allocating memory.\n");
		currentSong->next = nextSong;

		previousSong = currentSong;
		currentSong = nextSong;
	}

	lastSong = previousSong;
	lastSong->next = NULL;
	free(nextSong); 

	puts("\n");
}


	

int loadMusicCollection(void)
{
	datafile = fopen(filename, "rb");
	if(datafile != NULL)
	{
		firstSong = malloc(sizeof(*firstSong));
		if (firstSong == NULL)
			printf("Error allocating memory.\n");
		currentSong = firstSong;
		while(TRUE)
		{
			fread(currentSong, sizeof(*currentSong), 1, datafile);
			printf("File - %s\n", currentSong->file->d_name);

			if (previousSong == NULL) //This will be the first song.
				currentSong->previous = NULL;
			else
				currentSong->previous = previousSong;

			if (currentSong->next == NULL) //No more songs.
				break;

			previousSong = currentSong;
			nextSong = malloc(sizeof(*nextSong));
			if (nextSong == NULL)
						printf("Error allocating memory.\n");
			currentSong->next = nextSong;
			currentSong = nextSong;
		}
		fclose(datafile);
		printf("Data read from %s\n", filename);
		return(0);
	}
	else
	{
		printf("Error reading from %s\n", filename);
		return(1);
	}
}



int saveMusicCollection(void)
{
	if (firstSong == NULL)
		return(0); //No data to write.

	datafile = fopen(filename, "wb");

	if (datafile == NULL)
	{
		printf("Error writing to %s\n", filename);
		return(1);
	}

	currentSong = firstSong;
	while (currentSong != NULL)
	{
		fwrite(currentSong, sizeof(*currentSong), 1, datafile);
		printf("in save File - %s\n", currentSong->file->d_name);
		currentSong = currentSong->next;
	}
	fclose(datafile);
	printf("Data saved to %s\n", filename);

	return(0);
}



void readMusicCollection(void)
{
	

	currentSong = firstSong;
	while (currentSong != NULL)
	{

			printf("File - %s\n", currentSong->file->d_name);
			
		currentSong = currentSong -> next;
		puts("\n");
	}
}
  • 2 Contributors
  • 1 Reply
  • 83 Views
  • 6 Hours Discussion Span
  • Latest Post Latest Post by Narue
Member Avatar for Narue

Look at the referenced lines. Does strsong give you a char, or a pointer to char? The difference is significant, seeing as how you're trying to assign to a char variable.

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.